RFR(S): 8131129: Attempt to define a duplicate BMH$Species class

Michael Haupt michael.haupt at oracle.com
Mon Nov 30 10:51:34 UTC 2015


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.haupt at oracle.com>:
> 
> ... thanks a lot, Peter and Vladimir!
> 
> Best,
> 
> Michael
> 
>> Am 09.11.2015 um 13:42 schrieb Vladimir Ivanov <vladimir.x.ivanov at 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




More information about the core-libs-dev mailing list