[lilliput-jdk17u:lilliput] Integrated: 8310642: [Lilliput/JDK17] Remove unnecessary check in G1ParScanThreadState

[Roman Kennke]

On Thu, 22 Jun 2023 11:51:39 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

> G1ParScanThreadState::do_copy_to_survivor_space() has an unnecessary change against upstream:
> 
>  if (old_mark.is_marked()) {
>     // Already forwarded by somebody else, return forwardee.
>     return old->forwardee(old_mark);
>   }
> 
> All callers of this method already do the same, so this only adds code and possible affects performance for no reason.
> 
> Testing:
>  - [ ] hotspot_gc -UCOH
>  - [ ] hotspot_gc +UCOH
>  - [ ] tier1 -UCOH
>  - [ ] tier1 +UCOH
>  - [ ] tier2 -UCOH
>  - [ ] tier2 +UCOH

This pull request has now been integrated.

Changeset: 7cfcdff4
Author:    Roman Kennke <rkennke at openjdk.org>
URL:       https://git.openjdk.org/lilliput-jdk17u/commit/7cfcdff4400469d5167991f4059f31f71105ea25
Stats:     4 lines in 1 file changed: 0 ins; 4 del; 0 mod

8310642: [Lilliput/JDK17] Remove unnecessary check in G1ParScanThreadState

Reviewed-by: shade

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

PR: https://git.openjdk.org/lilliput-jdk17u/pull/42