RFR: updating supports_value_types() method and ACC_Value and ACC_Flattenable flags

harold seigel harold.seigel at oracle.com
Fri Mar 23 15:17:42 UTC 2018


Thanks Karen!

Harold


On 3/23/2018 11:15 AM, Karen Kinnear wrote:
> Harold,
>
> Looks good. Good catch.
>
> Karen
>
>> On Mar 23, 2018, at 10:28 AM, harold seigel <harold.seigel at oracle.com> wrote:
>>
>> Hi,
>>
>> Please review this change to fix method supports_value_types() in classFileParser.cpp and to ignore the ACC_Value and ACC_Flattenable flags if they are specified in older class file versions.  According to the JVM Spec for JDK-10 <https://docs.oracle.com/javase/specs/jvms/se10/html/index.html>:
>>
>>    All bits of the access_flags item not assigned in <table> are
>>    reserved for future use.  They should be set to zero in generated
>>    class files and *should be ignored by JVM implementations*.
>>
>> No new tests are needed because this is covered by existing JCK vm tests.
>>
>> Webrev: http://cr.openjdk.java.net/~hseigel/valueTypes_lworld.ACC_flags/webrev/index.html
>>
>> This webrev was tested with hotspot JTReg tests, many JDK JTReg tests, and with vm JCK tests.
>>
>> Thanks, Harold
>>



More information about the valhalla-dev mailing list