Integrated: 8306851: Move Method access flags

Coleen Phillimore coleenp at openjdk.org
Mon May 1 11:49:27 UTC 2023


On Tue, 25 Apr 2023 19:09:23 GMT, Coleen Phillimore <coleenp at 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 <coleenp at 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


More information about the hotspot-dev mailing list