RFR (S): 8027559: Decrease code size and templatizing in G1ParCopyClosure::do_oop_work
Stefan Karlsson
stefan.karlsson at oracle.com
Thu Feb 20 08:50:38 UTC 2014
On 2014-02-19 16:12, Thomas Schatzl wrote:
> Hi all,
>
> after the feedback from Stefan K. I split up the original change into
> multiple changesets
> (http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2014-February/009440.html).
>
> I agree, as Stefan indicated, it makes reviewing easier.
>
> Here is the first part.
>
> Changes included:
>
> - removed the unused _during_initial_mark, _mark_in_progress, _g1_rem
> members from G1ParClosureSuper
> - moved G1ParClosureSuper::_cm into G1ParCopyHelper as it is only used
> there
> - moved G1ParCopyClosure::mark_[forwarded_]object from G1ParCopyClosure
> into G1ParCopyHelper to avoid the multiple instantiations of these
> methods.
> - some minor cleanup
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8027559
>
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8027559/webrev.1/
Nice cleanup. Thanks for splitting this up into separate changesets.
Looks good.
StefanK
>
> Testing:
> jprt
>
> Thanks,
> Thomas
>
More information about the hotspot-gc-dev
mailing list