RFR (S): 8226232: Move merge heap roots code out from G1RemSetScanState
Kim Barrett
kim.barrett at oracle.com
Thu Jul 11 19:12:59 UTC 2019
> On Jul 10, 2019, at 11:30 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>
> Hi all,
>
> can I have reviews for this small change that moves code related to
> the merge heap roots phase out of G1RemSetScanState? This imho
> significantly increases readability of G1RemSetScanState.
>
> More minor cleanups to improve the code are planned, but I think it is
> good to take it in steps.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8226232
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8226232/webrev/
> Testing:
> hs-tier1-5
>
> Thanks,
> Thomas
Looks good.
One small item:
------------------------------------------------------------------------------
src/hotspot/share/gc/g1/g1RemSet.cpp
1166 {
1167 _scan_state->prepare_for_merge_heap_roots();
1168 }
Leftover extra scope nesting?
------------------------------------------------------------------------------
More information about the hotspot-gc-dev
mailing list