RFR: 8312262: Klass::array_klass() should return ArrayKlass pointer
Calvin Cheung
ccheung at openjdk.org
Thu Jul 27 21:00:53 UTC 2023
On Thu, 27 Jul 2023 20:16:17 GMT, Dean Long <dlong at openjdk.org> wrote:
>> This is a simple change to make array_klass() return ArrayKlass (the first dimension of TypeArrayKlass is a TypeArrayKlass so can't use ObjArrayKlass), higher_dimension is always an ObjArrayKlass and lower_dimension can be a TypeArrayKlass. The change removes some casts.
>> Tested with tier1 linux/macosx/windows on x86 and aarch64.
>
> src/hotspot/share/oops/arrayKlass.hpp line 62:
>
>> 60: ObjArrayKlass* higher_dimension() const { return _higher_dimension; }
>> 61: inline ObjArrayKlass* higher_dimension_acquire() const; // load with acquire semantics
>> 62: void set_higher_dimension(ObjArrayKlass* k) { _higher_dimension = k; }
>
> The set_higher_dimension() function seems to be unused. Should we remove it?
I'm using the function in the following PR https://github.com/openjdk/jdk/pull/14959.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15059#discussion_r1276813629
More information about the hotspot-dev
mailing list