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

Tobias Hartmann tobias.hartmann at oracle.com
Thu Feb 27 06:24:01 UTC 2020


Hi John,

On 21.02.20 11:23, John Rose wrote:
> Please have another look:
> 
> http://cr.openjdk.java.net/~jrose/values/atomic-8236522

The changes to valuetypenode.cpp/hpp are not required anymore after:
https://hg.openjdk.java.net/valhalla/valhalla/rev/b018052fc8b1

> (Old webrev is atomic-8236522.1)

http://cr.openjdk.java.net/~jrose/values/atomic-8236522.1/ only contains one file. Didn't you have a
tearing test in the first webrev thats should also be included in the new version?

Best regards,
Tobias


More information about the valhalla-dev mailing list