RFR: 8234688 [lworld] Supporting volatile semantic for flattenable fields

David Simms david.simms at oracle.com
Mon Nov 25 09:27:21 UTC 2019


I'm good with the change, nice to add a test, thanks for that.

Just a small comment: we could probably allow for the flattening for 
sizes that are naturally atomic for the H/W, i.e. x86_64 allows 64 bits 
? (Assuming correct alignment for "Copy::conjoint_memory_atomic" to 
consume).

/D


On 22/11/19 10:31 PM, Frederic Parain wrote:
> Please review this small changeset to support volatile flattenable fields.
> The short term solution is to not flatten these fields.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8234688
>
> Webrev:
> http://cr.openjdk.java.net/~fparain/volatile/webrev.00/
>
> Thank you,
>
> Fred




More information about the valhalla-dev mailing list