RFR: 8331194: NPE in ArrayCreationTree.java with -XX:-UseCompressedOops [v5]

[Cesar Soares Lucas]

On Tue, 16 Jul 2024 08:27:30 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Cesar Soares Lucas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix test package path.
>
> src/hotspot/share/opto/output.cpp line 990:
> 
>> 988:     }
>> 989: 
>> 990:     ObjectValue* other = (ObjectValue*) sv_for_node_id(objs, n->_idx);
> 
> Is the cast here necessary? I see them generally in the file... but not sure why.
> 
> ObjectValue*
> PhaseOutput::sv_for_node_id(GrowableArray<ScopeValue*> *objs, int id) {

Thank you @eme64 , @vnkozlov for reviewing. I created this RFE to remove the unnecessary casts: https://bugs.openjdk.org/browse/JDK-8336495

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20087#discussion_r1680000070