[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