[8u] Request for approval: Backport of 8144487 and 8145754

Tobias Hartmann tobias.hartmann at oracle.com
Mon Jan 4 10:04:06 UTC 2016


Thanks, David! I will push this to 8u-dev as soon as I get a peer review for the backport.

Best,
Tobias

On 04.01.2016 10:56, david buck wrote:
> approved for backport to 8u-dev
> 
> Thank you for adding the review links.
> 
> Cheers,
> -Buck
> 
> On 2016/01/04 18:30, Tobias Hartmann wrote:
>> Hi David,
>>
>> sure, I included the links to the code review:
>>
>> 8144487: PhaseIdealLoop::build_and_optimize() must restore major_progress flag if skip_loop_opts is true
>> https://bugs.openjdk.java.net/browse/JDK-8144487
>> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2015-December/020503.html
>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/21689239c407
>>
>> 8145754: PhaseIdealLoop::is_scaled_iv_plus_offset() does not match AddI
>> https://bugs.openjdk.java.net/browse/JDK-8145754
>> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2015-December/020502.html
>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/0e9d64117522
>>
>> Thanks,
>> Tobias
>>
>> On 04.01.2016 09:35, david buck wrote:
>>> Hi Tobias!
>>>
>>> Would you please include links to the code review threads on mail.openjdk.java.net?
>>>
>>> [ JDK 8 Updates: Push Approval Request Template ]
>>> http://openjdk.java.net/projects/jdk8u/approval-template.html
>>>
>>> Cheers,
>>> -Buck
>>>
>>> On 2016/01/04 17:15, Tobias Hartmann wrote:
>>>> Hi,
>>>>
>>>> please approve and review the following backports to 8u.
>>>>
>>>> 8144487: PhaseIdealLoop::build_and_optimize() must restore major_progress flag if skip_loop_opts is true
>>>> https://bugs.openjdk.java.net/browse/JDK-8144487
>>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/21689239c407
>>>>
>>>> 8145754: PhaseIdealLoop::is_scaled_iv_plus_offset() does not match AddI
>>>> https://bugs.openjdk.java.net/browse/JDK-8145754
>>>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/0e9d64117522
>>>>
>>>> Nightly testing showed no problems and the changes apply cleanly to 8u-dev.
>>>>
>>>> Thanks,
>>>> Tobias
>>>>


More information about the hotspot-compiler-dev mailing list