[lworld] RFR: 8374882: [lworld] Update hierarchies in oopsHierarchy.hpp
Joel Sikström
jsikstro at openjdk.org
Fri Jan 9 14:42:36 UTC 2026
On Fri, 9 Jan 2026 14:08:16 GMT, Frederic Parain <fparain at openjdk.org> 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
More information about the valhalla-dev
mailing list