[aarch64-port-dev ] 8078263: AARCH64: Volatile fields should be accessed with ldar and stlr

[Andrew Dinn]

On 23/04/15 17:54, Andrew Haley wrote:
> On 04/21/2015 02:43 PM, Andrew Dinn wrote:
>> On 21/04/15 13:53, Andrew Haley wrote:
>>> On 04/20/2015 10:03 AM, Andrew Dinn wrote:
>>>> The following webrev is submitted for review (n.b. I am not an OpenJDK
>>>> committer so the patch does not yet have an associated JIRA).
>>>
>>> The patch looks good to me.
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8078263
>>
>> A new webrev with the commit in standard format (listing JIRA#, sumamry
>> and reviewer) and placed at:
>>
>>   http://cr.openjdk.java.net/~adinn/8078263/webrev.01/
> 
> I'm surprised this got past jcheck:

Ah yes, jcheck . . . ahem [enables jcheck]

New webrev which passes jcheck

  http://cr.openjdk.java.net/~adinn/8078263/webrev.02/

regards,


Andrew Dinn
-----------
Senior Principal Software Engineer
Red Hat UK Ltd
Registered in UK and Wales under Company Registration No. 3798903
Directors: Michael Cunningham (USA), Matt Parson (USA), Charlie Peters
(USA), Michael O'Neill (Ireland)