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

Thomas Schatzl thomas.schatzl at oracle.com
Wed Feb 22 10:14:01 UTC 2017


Hi Erik,

On Wed, 2017-02-22 at 10:46 +0100, Erik Helin wrote:
> 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!

  thanks for your review.

Thomas




More information about the hotspot-gc-dev mailing list