RFR: JDK-8240315: Shenandoah: Rename ShLBN::get_barrier_strength()
Roman Kennke
rkennke at redhat.com
Fri Mar 6 12:25:28 UTC 2020
Hi Aleksey,
>> 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"?
Ok.
> *) Given how all uses invert the returned value, maybe we should revert the meaning to be
> "is_redundant"?
Right. Done.
> *) 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.
Right. I've updated it.
http://cr.openjdk.java.net/~rkennke/JDK-8240315/webrev.01/
Ok now?
Roman
More information about the shenandoah-dev
mailing list