RFR (S) 8175887: C1 value numbering handling of Unsafe.get*Volatile is incorrect

[Aleksey Shipilev]

Hi again,

If you see no troubles in nightly runs, can you propose this to 8u?
I can propose myself, if you want.

-Aleksey

On 03/01/2017 02:41 PM, Vladimir Ivanov wrote:
> Pushed:
>   http://hg.openjdk.java.net/jdk9/hs/hotspot/rev/2ff05d967fb2
> 
> Best regards,
> Vladimir Ivanov
> 
> On 3/1/17 12:01 AM, Aleksey Shipilev wrote:
>> On 02/28/2017 10:01 PM, Vladimir Ivanov wrote:
>>>
>>>>>  http://cr.openjdk.java.net/~vlivanov/shade/8175887/
>>>>
>>>> The cleanup look good, but are you sure AssertionError would get reported from
>>>> the thread, and it would not silently die? The original test exited hard to
>>>> handle this.
>>>
>>> Good point. Reverted that part. (Updated the webrev in place.)
>>
>> Thumbs up. Thanks for handling this!
>>
>> -Aleksey
>>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20170302/9c27e53c/signature.asc>