RFR: Add missing case for CAE to ShLRB::needs_barrier_impl()

Aleksey Shipilev shade at redhat.com
Fri Feb 22 11:42:57 UTC 2019


On 2/22/19 12:40 PM, Roman Kennke wrote:
> Am 22. Februar 2019 12:24:57 MEZ schrieb Aleksey Shipilev <shade at redhat.com>:
>> On 2/22/19 12:23 PM, Roman Kennke wrote:
>>> http://cr.openjdk.java.net/~rkennke/lrb-cae-2/webrev.00/
>> 
>> But wait, how do we even get these nodes with Shenandoah enabled? Isn't that basically a bug 
>> somewhere else?
> 
> We replace the Shenandoah variants by the barrier-free variants when cmpval input is known NULL,
> in which case we don't need the barrier loop, and it's fairly common, too.
I see. Looks good then!

-Aleksey



More information about the shenandoah-dev mailing list