On Mon, 24 Aug 2026 15:41:18 GMT, Coleen Phillimore <[email protected]> wrote:
>> More non-asserts:
>>
>> // Update hierarchy. This is done before the new klass has been added to the
>> SystemDictionary. The Compile_lock
>> // is grabbed, to ensure that the compiler is not using the class hierarchy.
>> void InstanceKlass::add_to_hierarchy(JavaThread* current) {
>
> The usual path calls add_to_hierarchy() first so this should be okay. If it
> mattered, the usual case in system dictionary would more likely expose any
> dependencies on these states. I think this is okay.
I think this is also single threaded at startup, so also doesn't matter which
order this is in.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32218#discussion_r3845095054