for review: 8236522: "always atomic" modifier for inline classes to enforce atomicity
John Rose
john.r.rose at oracle.com
Mon Jan 13 20:57:57 UTC 2020
On Jan 13, 2020, at 12:08 AM, Tobias Hartmann <tobias.hartmann at oracle.com> 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?
> http://hg.openjdk.java.net/jdk/jdk/file/f7edb9ca045c/src/hotspot/share/opto/macro.cpp#l1406
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