[lworld] RFR: 8267710: Hook AlwaysAtomicAccesses to primitive classes atomicity rules

Frederic Parain fparain at openjdk.java.net
Tue May 25 21:51:18 UTC 2021


On Tue, 25 May 2021 17:27:44 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> `AlwaysAtomicAccesses` was added in JDK 9 to aid the research in the costs of requiring the always-atomic accesses. It would be nice to hook it up to current Valhalla code that handles explicitly declared atomic primitive classes.
> 
> Testing:
>  - [x] ad-hoc jcstress tests with lworld and `-XX:(-|+)AlwaysAtomicAccesses`

Looks good to me.

Fred

You can push now.
"[lworld] [lw3]" in the title of CR are more of a custom than a rule.
What is important is to have the labels (I've added them to the CR).

Could you add the same fix to method FieldLayoutBuilder::inline_class_field_sorting()?
Thank you.

Fred

-------------

Marked as reviewed by fparain (Committer).

PR: https://git.openjdk.java.net/valhalla/pull/427


More information about the valhalla-dev mailing list