On Mon, 5 Jan 2026 13:13:05 GMT, Tobias Hartmann <[email protected]> wrote:
>> Quan Anh Mai has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains six commits:
>>
>> - wrong value is passed as flat
>> - remove redundant changes
>> - Merge branch 'lworld' into ciRefArray
>> - Revert "can only infer klass if exact"
>>
>> This reverts commit be6143624d30c9ebfa560dd8161aa70270c68d24.
>> - can only infer klass if exact
>> - Refactor ciObjArrayKlass to align with the VM type
>
> src/hotspot/share/opto/type.cpp line 3700:
>
>> 3698: _is_ptr_to_narrowoop = UseCompressedOops &&
>> ::is_reference_type(field_bt);
>> 3699: } else if (klass()->is_obj_array_klass()) {
>> 3700: _is_ptr_to_narrowoop = UseCompressedOops;
>
> `UseCompressedOops` is always true here, right? It's checked in line 3685.
> Also above.
You are right, I have removed those check.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/1739#discussion_r2664699458