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

Reply via email to