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

[Seán Coffey]

Please get a peer review before pushing to the jdk8u-dev forest. Looks 
like the bug report also needs a noreg label [1]

Approved.

[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
>