[8u-dev] Request for approval: Backport of 8011858

[Zoltán Majó]

Hi Seán,


On 09/14/2015 04:54 PM, Seán Coffey wrote:
> Please get a peer review before pushing to the jdk8u-dev forest.

I will do so.

> Looks like the bug report also needs a noreg label [1]

I added the "noreg-hard" label.

>
> Approved.

Thank you for the review!

Best regards,


Zoltán

>
> [1] http://openjdk.java.net/guide/changePlanning.html#noreg
>
> Regards,
> Sean.
>
> On 14/09/15 15:31, Zoltán Majó wrote:
>> Hi,
>>
>>
>> I would like to request the backport of the fix for JDK-8011858 to 
>> 8u-dev. The fix can help reducing the memory usage of the C2 compiler 
>> and is therefore needed to fix JDK-8129847.
>>
>> Original (9) bug: https://bugs.openjdk.java.net/browse/JDK-8011858
>> Original changeset: 
>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/af60f1cb36f2
>>
>> Unfortunately, the fix does not apply cleanly, as two affected files 
>> are different in 9 and in 8u-dev. Here are the changes that I did not 
>> (could not) apply to the 8u-dev tree:
>> - http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/af60f1cb36f2#l6.8
>> - http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/af60f1cb36f2#l9.8
>>
>> Webrev: http://cr.openjdk.java.net/~zmajo/8011858_8u/webrev.00/
>>
>> Testing:
>> - the fix was pushed into 9 on Aug 11 and did not cause any nightly 
>> failures;
>> - all JPRT tests pass;
>> - all JTREG tests pass that pass with the unmodified VM.
>>
>> Thank you and best regards,
>>
>>
>> Zoltan
>>
>