On Fri, 13 Mar 2026 09:17:09 GMT, Stefan Karlsson <[email protected]> wrote:

>> The current lworld branch has code that visits the 
>> ObjArrayKlass::_element_klass. The element klass and the array klass are 
>> both allocated in the same CLD, so there's no need for the GC to spend extra 
>> work also trying to keep the element klass alive. 
>> 
>> The patch also contains some tiny reverts and cleanups that I saw while 
>> diffing the GC code in the lworld branch.
>> 
>> The patch depends on the #2207 so that needs to get reviewed before this can 
>> be integrated.
>> 
>> Tested with tier1-3
>
> Stefan Karlsson has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains 14 commits:
> 
>  - Merge remote-tracking branch 'valhalla/lworld' into 
> 8379847_remove_element_klass_visit
>  - 8379847: [lworld] Remove added visit of array element klass from GC 
> visitors
>  - Merge remote-tracking branch 'valhalla/lworld' into 
> lworld_array_klass_cleanups
>  - faKlass => fak
>  - Stray whitespaces
>  - Restore jniCheck.cpp
>  - Array cleanups
>  - Retype _objectArrayKlass
>  - allocate_instance
>  - Small cleanups
>  - ... and 4 more: 
> https://git.openjdk.org/valhalla/compare/dde95fd7...ff0b3a25

Thanks for the reviews!

-------------

PR Comment: https://git.openjdk.org/valhalla/pull/2218#issuecomment-4054174835

Reply via email to