[lworld] RFR: 8279428: [lworld] Revalue ACC_PRIMITIVE to be 0x800 to align with draft JVMS
Aggelos Biboudis
duke at openjdk.java.net
Fri Jan 7 09:26:43 UTC 2022
On Fri, 7 Jan 2022 05:48:17 GMT, Srikanth Adayapalam <sadayapalam at openjdk.org> wrote:
>> Thank you for the detailed explanation.
>>
>>> In the new code, again LocalClassFlags is missing ACC_PRIMITIVE, but things work just fine but by accident - only because ACC_PRIMITIVE shares the same value with ACC_STRICT !
>>
>> I am still a bit confused because I don't see any uses of ACC_STRICT but only STRICT_FP at that level (which is indeed 1<<11, 0x0800). So in my mind it shares the same value with STRICT_FP not ACC_STRICT. I will fully get it with your upcoming revision 👍 💯
>>
>> Thanks again for the explanation!
>
> But ACC_STRICT and STRICT_FP symbolize/represent the same and carry the same value. (The latter is javac's internal symbol.)
Got it, right!
-------------
PR: https://git.openjdk.java.net/valhalla/pull/593
More information about the valhalla-dev
mailing list