RFR: 8266530: HotSpot changes for JEP 306 [v2]

Joe Darcy darcy at openjdk.java.net
Thu May 6 17:40:33 UTC 2021


On Thu, 6 May 2021 13:34:32 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> Joe Darcy has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Respond to review feedback.
>
> src/hotspot/share/classfile/classFileParser.cpp line 4802:
> 
>> 4800:           // those flags are illegal irrespective of ACC_ABSTRACT being set or not.
>> 4801:           (is_abstract &&
>> 4802:            (is_private || is_static || (major_gte_17 ? false : is_strict)))) {
> 
> Lines 4802 and 4830 would be simpler if changed to:
>    (is_private || is_static || (!major_gte_17  && is_strict)))) {

Suggested changes here and in the test made; thanks.

-------------

PR: https://git.openjdk.java.net/jdk/pull/3891


More information about the hotspot-runtime-dev mailing list