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

John Rose john.r.rose at
Mon Jan 13 20:57:57 UTC 2020

On Jan 13, 2020, at 12:08 AM, Tobias Hartmann <tobias.hartmann at> wrote:
> Hi John,
> On 24.12.19 22:36, John Rose wrote:
>> This uncovered a missing StoreStore barrier in C2, which I also fixed.
> Do you know why that barrier is not added in PhaseMacroExpand::expand_allocate_common?

I assume one of the guard conditions disables it.
Maybe the test of does_not_escape_thread?
— John

More information about the valhalla-dev mailing list