RFR: 8271579: G1: Move copy before CAS in do_copy_to_survivor_space [v4]

[Thomas Schatzl]

On Tue, 17 Aug 2021 01:45:51 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Propose to move copy before CAS in do_copy_to_survivor_space, as we found this will improve G1 performance. Specjbb shows 3.7% in critical on aarch64, no change in max.
>> 
>> After this change copy_to_survivor in G1 is also aligned with PS's copy_to_survivor.
>
> Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Closes JDK-8272070: simplify age calculation

Just for clarification: the default requirements for a change in the gc-team to integrate is to have at least two "r"eviewers (note the lowercase r - those are people that approved it regardless of role), at least one of which must be a "R"eviewer, i.e. has the openjdk reviewer role (and there are no other open questions/comments).

So that formal requirement had already been fulfilled by me and Albert without Albert being a "R"eviewer.

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

PR: https://git.openjdk.java.net/jdk/pull/4983