On Tue, 25 Apr 2023 19:09:23 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:
> This change moves the flags from AccessFlags to either ConstMethodFlags or > MethodFlags, depending on whether they are set at class file parse time, > which makes them essentially const, or at runtime, which makes them needing > atomic access. > > This leaves AccessFlags int size because Klass still has JVM flags that are > more work to move, but this change doesn't increase Method size. I didn't > remove JVM_RECOGNIZED_METHOD_MODIFIERS with this change since there are > several of these in other places, and with this change the code is benign. > > Tested with tier1-6, and some manual verification of printing. This pull request has now been integrated. Changeset: 316d303c Author: Coleen Phillimore <cole...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/316d303c1da550c9589c9be56b65650964e3886b Stats: 781 lines in 27 files changed: 316 ins; 297 del; 168 mod 8306851: Move Method access flags Reviewed-by: cjplummer, dholmes, dnsimon, matsaave, fparain ------------- PR: https://git.openjdk.org/jdk/pull/13654
