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

Tobias Hartmann tobias.hartmann at oracle.com
Mon Jan 13 08:08:17 UTC 2020


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

Thanks,
Tobias


More information about the valhalla-dev mailing list