RFR (S): 8162104: Use in_cset() instead of obj_in_cs()

Erik Helin erik.helin at oracle.com
Wed Feb 22 09:46:38 UTC 2017


On 02/22/2017 10:35 AM, Thomas Schatzl wrote:
> Hi all,
>
>   could I have reviews for this small cleanup that removes the use of
> an obsolete method?
>
> G1CollectedHeap::obj_in_cs() provides the same functionality as
> G1CollectedHeap::is_in_cset(), except being slower. In fact, we had
> asserts in the is_in_cset() code that used obj_in_cs.
>
> Obj_in_cs has only been used in asserts too.
>
> I see no need to maintain two versions of the same thing, hence the
> removal.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8162104
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8162104/webrev/

Thanks for cleaning this up! Looks good, Reviewed!

Erik

> Testing:
> jprt
>
> Thanks,
>   Thomas
>



More information about the hotspot-gc-dev mailing list