RFR: JDK-8240315: Shenandoah: Rename ShLBN::get_barrier_strength()

Aleksey Shipilev shade at redhat.com
Thu Mar 5 16:57:01 UTC 2020


On 3/5/20 5:42 PM, Roman Kennke wrote:
> I would like to rename
> ShenandoahLoadReferenceBarrierNode::get_barrier_strength() to
> is_lrb_useful(). See:
> 
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8240315
> Webrev:
> http://cr.openjdk.java.net/~rkennke/JDK-8240315/webrev.00/

*) "lrb" seems redundant, better be "is_useful"?

*) Given how all uses invert the returned value, maybe we should revert the meaning to be
"is_redundant"?

*) This comment is outdated:

3198   // Look for strongest strength: go over nodes looking for STRONG ones.
3199   // Stop once we encountered STRONG. Otherwise, walk until we ran out of nodes,
3200   // and then the overall strength is NONE.

-- 
Thanks,
-Aleksey



More information about the shenandoah-dev mailing list