for review: 8236522: "always atomic" modifier for inline classes to enforce atomicity

Tobias Hartmann tobias.hartmann at oracle.com
Wed Feb 12 15:36:59 UTC 2020


Hi John,

I've just sent the RFR for 8238780 which will fix these issues. I found one additional place where a
StoreStore barrier was missing.

Thanks,
Tobias

On 22.01.20 21:20, John Rose wrote:
> On Jan 20, 2020, at 6:54 AM, Tobias Hartmann <tobias.hartmann at oracle.com
> <mailto:tobias.hartmann at oracle.com>> wrote:
>>
>>>
>>> How did it reproduce for you? It would be nice to have a regression test.
>>
>> Okay, -XX:+StressLCM did the trick :)
> 
> Thanks for taking it out for a spin.
> 
> The jtreg test ValueTearing.java failed reliably (P ~ 50%)
> on my MacBook.  Debugging that led me to add that S-S barrier.
> 
> — John


More information about the valhalla-dev mailing list