On Thu, 16 Apr 2026 05:24:36 GMT, David Holmes <[email protected]> wrote:
>> src/hotspot/share/oops/instanceKlass.cpp line 3706:
>>
>>> 3704:
>>> 3705: #if !defined(PRODUCT) || INCLUDE_JVMTI
>>> 3706: void InstanceKlass::print_class_flags(outputStream* st) const {
>>
>> This is incomplete. If the class is an inner class then additional access
>> flags are possible (private, protected, static).
>> EDIT: Hmm jvm_constants.h does not recognise this either via
>> `JVM_RECOGNIZED_CLASS_MODIFIERS`. Not sure how this should be handled.
>
> Also do we need to handle ACC_MODULE, or do we not actually create an
> `instanceKlass` for those?
On jdk side we never have Class for modules. These classfiles are exclusively
handled by Java code in ModuleDescriptor I think?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30746#discussion_r3093010444
