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

[Aleksey Shipilev]

Thank you!

-Aleksey

On 03/09/2017 05:50 PM, Vladimir Ivanov wrote:
> Pushed: http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/0e53cdb9fc2a
> 
> Best regards,
> Vladimir Ivanov
> 
> On 3/8/17 4:24 PM, Aleksey Shipilev wrote:
>> (should have copied hs-comp-dev@, sorry)
>>
>> Need a sponsor, thanks!
>>
>> 8u changeset, if you need one:
>>   http://cr.openjdk.java.net/~shade/8175887/hs-8175887-8u.changeset
>>
>> -Aleksey
>>
>> On 03/06/2017 10:36 AM, Aleksey Shipilev wrote:
>>> Thanks Rob!
>>>
>>> Vladimir Ivanov, could you sponsor this one too, please?
>>>
>>> -Aleksey
>>>
>>> On 03/04/2017 12:09 AM, Rob McKenna wrote:
>>>> Approved, please work with the compiler team to find an appropriate
>>>> sponsor.
>>>>
>>>>     -Rob
>>>>
>>>> On 03/03/17 06:48, Aleksey Shipilev wrote:
>>>>> Hi,
>>>>>
>>>>> Please approve the backport of this important bugfix to 8u:
>>>>>
>>>>> Bug:
>>>>>   https://bugs.openjdk.java.net/browse/JDK-8175887
>>>>>
>>>>> JDK 9 changeset:
>>>>>   http://hg.openjdk.java.net/jdk9/hs/hotspot/rev/2ff05d967fb2
>>>>>
>>>>> Compiler folks agree with backport, and nightlies seem clean:
>>>>>  
>>>>> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-March/025767.html
>>>>>
>>>>>
>>>>> JDK 8u webrev:
>>>>>   http://cr.openjdk.java.net/~shade/8175887/webrev.04.8u/
>>>>>
>>>>> Product change applies cleanly, but test needs to change j.i.m.Unsafe to
>>>>> s.m.Unsafe.
>>>>>
>>>>> Testing: Linux x86_64/release hotspot/test/compiler/c1 before/after the fix.
>>>>>
>>>>> Need a sponsor to push.
>>>>>
>>>>> Thanks,
>>>>> -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/20170309/765f6902/signature.asc>