RFR: 8309717: C2: Remove Arena::move_contents usage

Johan Sjölen jsjolen at openjdk.org
Wed Jun 14 12:34:53 UTC 2023


On Fri, 9 Jun 2023 10:17:46 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

> Hi,
> 
> Instead of using `Arena::move_contents` we can just see the arena swap as a form of double buffering, reducing this to a pointer swap and a clear. This allows us to remove `Arena::move_contents`, cleaning up the arena code.
> 
> Since this requires allocating another pointer for `Compile`, I took the time to move some members around in order to reduce the padding. This means that this patch does *not* introduce a size change for `Compile`.
> 
> I'm currently running tier1-3 tests.
> 
> Thanks for considering this,
> Johan

Thanks Tobias, I took your changes and applied them.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/14391#issuecomment-1591097852


More information about the hotspot-dev mailing list