Hello, FYI, I've requested backport approval for this fix to 8u. http://mail.openjdk.java.net/pipermail/jdk8u-dev/2015-November/004540.html
Best, Michael > Am 09.11.2015 um 16:28 schrieb Michael Haupt <michael.ha...@oracle.com>: > > ... thanks a lot, Peter and Vladimir! > > Best, > > Michael > >> Am 09.11.2015 um 13:42 schrieb Vladimir Ivanov >> <vladimir.x.iva...@oracle.com>: >> >> Looks good! >> >> Best regards, >> Vladimir Ivanov >> >> On 11/9/15 3:16 PM, Peter Levart wrote: >>> Hi all, >>> >>> Thanks for analysis, reviews and discussion that hopefully beheaded this >>> hydra. >>> >>> I added an assert to method setSpeciesDataToConcreteBMHClass() that >>> verifies the presence of @Stable annotation on the SPECIES_DATA field of >>> the generated class. It caught a bug I made when I specified a binary >>> class name "java/lang/invoke/Stable" in FieldVisitor.visitAnnotation() >>> instead of type signature "Ljava/lang/invoke/Stable;" >>> >>> The consequence was not drastic - the field was just not being >>> interpreted by VM as @Stable. >>> >>> With that adjustment and successful re-run of jtreg tests, I ask for >>> re-confirmation to push the following: >>> >>> http://cr.openjdk.java.net/~plevart/jdk9-dev/BMH.race/webrev.07/ >>> >>> Regards, Peter >>> >>> On 11/06/2015 01:31 PM, Vladimir Ivanov wrote: >>>> Peter, >>>> >>>>> >>>>> http://cr.openjdk.java.net/~plevart/jdk9-dev/BMH.race/webrev.06/ >>>>> >>>> >>>> Looks really good! Reviewed. >>>> >>>>> - the SPECIES_DATA field in generated class can not be static final when >>>>> it is initialized out of <clinit>. Just static. It was just static >>>>> before when it was initialized in <clinit> and could be static final. We >>>>> can make it @Stable at least. >>>>> >>>> Not sure it is on hot path, but I'm all for marking the field as @Stable >>>> >>>>> - there's no need for empty <clinit> method now. Removed. >>>>> >>>>> Basic java/lang/invoke jtreg tests pass. >>>> >>>> Best regards, >>>> Vladimir Ivanov >>> > > -- > > <http://www.oracle.com/> > Dr. Michael Haupt | Principal Member of Technical Staff > Phone: +49 331 200 7277 | Fax: +49 331 200 7561 > Oracle Java Platform Group | LangTools Team | Nashorn > Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany > <http://www.oracle.com/commitment> Oracle is committed to developing > practices and products that help protect the environment > -- <http://www.oracle.com/> Dr. Michael Haupt | Principal Member of Technical Staff Phone: +49 331 200 7277 | Fax: +49 331 200 7561 Oracle Java Platform Group | LangTools Team | Nashorn Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany <http://www.oracle.com/commitment> Oracle is committed to developing practices and products that help protect the environment