Integrated: 8265450: Merge PreservedMarksSet::restore code paths

[Thomas Schatzl]

On Tue, 20 Apr 2021 08:39:56 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   can I have reviews for this refactoring  that merges parallel and serial preserved marks restoration during evacuation failure into a single code path, simply by having the serial path call the parallel path in the VM thread.
> 
> This just reduces code and avoids the potential bugs when updating one path but not the other (which already happened).
> 
> The execution difference should not really matter for Serial GC where the serial path is executed:
> - we are talking about the evacuation failure path
> - there are likely a lot of preserved marks (random writes to the heap) that offset these two additional calls and some tiny superfluous member initialization.
> 
> Testing: tier1
> 
> Thanks,
>   Thomas

This pull request has now been integrated.

Changeset: 325edbc3
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/325edbc3
Stats:     49 lines in 1 file changed: 11 ins; 14 del; 24 mod

8265450: Merge PreservedMarksSet::restore code paths

Reviewed-by: sjohanss, iwalulya

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

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