On Fri, 9 Jan 2026 14:08:16 GMT, Frederic Parain <[email protected]> wrote:

>> I don't think so:
>> `class refArrayOopDesc : public arrayOopDesc {`
>
> But :
> `class RefArrayKlass : public ObjArrayKlass
> `
> So there's an unconsistency here, `refArrayOopDesc` should be a subclass of 
> `objArrayOopDesc`.

I agree that it would be clearer to make `refArrayOopDesc` inherit from 
`objArrayOopDesc`. I'll run some tests to see if it's a simple "switch" to 
inherit from `objArrayOopDesc` instead. If it's a trivial change I can add it 
to this PR and change the hierarchy in oopsHierarchy.hpp, otherwise I suggest 
we should handle that in a follow-up.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/1874#discussion_r2676427403

Reply via email to