[8u60] Bulk request for approval: 8063137, 8069591, 8068915, 8057919, 8074548, 8062280, 8071788, 8077054

Seán Coffey sean.coffey at oracle.com
Thu Apr 16 20:11:15 UTC 2015


Vladimir,

comments inline..

On 16/04/2015 20:51, Vladimir Ivanov wrote:
> Please, approve backports of the following fixes into 8u60:
>
> (1) 8063137: "Never taken branches should be pruned when GWT 
> LambdaForms are shared"
> https://bugs.openjdk.java.net/browse/JDK-8063137
> http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/d8bbc2fb7df8
this needs a no-reg label.
>
> (2) 8069591: "Customize LambdaForms which are invoked using 
> MH.invoke/invokeExact"
> https://bugs.openjdk.java.net/browse/JDK-8069591
> http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/86fbdebd98af
this needs a no-reg label
>
> (3) 8068915: "C2: uncommon trap w/ Reason_speculate_class_check causes 
> performance regression due to continuous deoptimizations"
> https://bugs.openjdk.java.net/browse/JDK-8068915
> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/ea802a4750aa
Ideally, this should go to the hotspot team forest. That forest doesn't 
require integration approval. If you've the fix committed as part of 
this push, I've no strong objections to jdk8u-dev as long as all jdk and 
hotspot JPRT tests are green. It requires a noreg label
>
> (4) 8057919: "Class.getSimpleName() should work for non-JLS compliant 
> class names"
> https://bugs.openjdk.java.net/browse/JDK-8057919
> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/f90702a3d522
> http://hg.openjdk.java.net/jdk9/hs-comp/jdk/rev/63e229f7f822
>
> (5) 8074548: "Never-taken branches cause repeated deopts in MHs.GWT case"
> https://bugs.openjdk.java.net/browse/JDK-8074548
> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/1aa6df528006
Like 8068915 - ideally it should go to the 8u hs-dev team forest first.
>
> (6) 8062280: "C2: inlining failure due to access checks being too strict"
> https://bugs.openjdk.java.net/browse/JDK-8062280
> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/528423a8c09f
Should go to hs-dev team forest
>
> (7) 8071788: "CountingWrapper.asType() is broken"
> https://bugs.openjdk.java.net/browse/JDK-8071788
> http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/2dc4afee9a85
>
> (8) 8077054: "DMH LFs should be customizeable"
> https://bugs.openjdk.java.net/browse/JDK-8077054
> http://hg.openjdk.java.net/jdk9/hs-comp/jdk/rev/16381a5f0959
Needs a noreg label.

Consider these approved but please seek guidance from Alejandro on 
whether some of the hotspot specific fixes should enter 8u codeline via 
the 8u hotspot team forest.

regards,
Sean.
>
> The fixes apply cleanly.
>
> Thanks!
>
> Best regards,
> Vladimir Ivanov



More information about the jdk8u-dev mailing list