RFR(S): 8131129: Attempt to define a duplicate BMH$Species class
Michael Haupt
michael.haupt at oracle.com
Mon Nov 9 15:28:05 UTC 2015
... 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
More information about the core-libs-dev
mailing list