RFR(S): 8139434: Unify GenRemSet and CardTableRS
Jesper Wilhelmsson
jesper.wilhelmsson at oracle.com
Mon Oct 12 20:48:54 UTC 2015
Looks good in general.
* In several files you have removed the include of genRemSet.hpp but only i one
place you add the include of cardTableRS.hpp. I think all source files that uses
CardTableRS should include cardTableRS.hpp
* Minor indentation error at the end of the line in vmStructs.cpp
Thanks,
/Jesper
Den 12/10/15 kl. 15:33, skrev David Lindholm:
> Hi,
>
> Please review this patch that unifies GenRemSet and CardTableRS, since
> CardTableRS is the only class that implements GenRemSet.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8139434
> Webrev: http://cr.openjdk.java.net/~david/JDK-8139434/webrev.00/
>
> Testing: Passed JPRT
>
>
> Thanks,
> David
More information about the hotspot-gc-dev
mailing list