[lworld] RFR: 8279428: [lworld] Revalue ACC_PRIMITIVE to be 0x800 to align with draft JVMS
Aggelos Biboudis
duke at openjdk.java.net
Thu Jan 6 15:33:45 UTC 2022
On Thu, 6 Jan 2022 09:54:17 GMT, Srikanth Adayapalam <sadayapalam at openjdk.org> wrote:
> Change ACC_PRIMITIVE to be 0x800 across components sources and tests
src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassWriter.java line 1551:
> 1549: flags = adjustFlags(c.flags() & ~DEFAULT);
> 1550: if ((flags & PROTECTED) != 0) flags |= PUBLIC;
> 1551: flags = flags & (ClassFlags | ACC_PRIMITIVE) & ~STRICTFP;
As far as I can understand, `ACC_PRIMITIVE` was removed from here because it is driven by the relevant if-clause in relation to `PRIMITIVE_CLASS` in `adjustFlags` which makes the code clearer at this point. Is my assumption correct?
-------------
PR: https://git.openjdk.java.net/valhalla/pull/593
More information about the valhalla-dev
mailing list