Integrated: 8309717: C2: Remove Arena::move_contents usage

[Johan Sjölen]

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

This pull request has now been integrated.

Changeset: 4c0e1642
Author:    Johan Sjölen <jsjolen at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/4c0e164238458e0e95770a855ba84bb265ff0397
Stats:     53 lines in 5 files changed: 19 ins; 22 del; 12 mod

8309717: C2: Remove Arena::move_contents usage

Reviewed-by: thartmann, kvn

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

PR: https://git.openjdk.org/jdk/pull/14391