Quantcast

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

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

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

Aleksey Shipilev-4
(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
>>>
>


signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

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

Vladimir Ivanov
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
>>>>
>>
>
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

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

Aleksey Shipilev-4
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
>>>>>
>>>
>>


signature.asc (836 bytes) Download Attachment
Loading...