On Thu, 12 Feb 2026 19:49:48 GMT, Matias Saavedra Silva <[email protected]> 
wrote:

> Different array layouts result in distinct array klass metadata which are 
> stored in a linked list with the base ObjArrayKlass acting as the head. 
> Multiple layouts are currently uncommon, so this assert was sufficient:
> 
> `assert(objarray_k == k || objarray_k->next_refined_array_klass() == k) 
> failed: must be`
> 
> This assert is not correct since `k` could refer to another refined array 
> klass further down the linked list. This patch corrects this assert by adding 
> a find method to iterate through the linked list in case there are multiple 
> layouts for the same type. A new test was added to catch this failure. 
> Verified with tier 1-5 tests.

This looks good!

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

Marked as reviewed by coleenp (Committer).

PR Review: 
https://git.openjdk.org/valhalla/pull/2095#pullrequestreview-3793609729

Reply via email to