RFR: 8267904: C2 crash when compile negative Arrays.copyOf length after loop [v5]

[Hui Shi]

On Thu, 3 Jun 2021 15:38:03 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> Hui Shi has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove unecessary gvn set_type_bottom and record_for_igvn when creating CastIINode after array allocation
>
> src/hotspot/share/opto/library_call.cpp line 4468:
> 
>> 4466: #endif
>> 4467:         C->gvn_replace_by(init_out, alloc_length);
>> 4468:         record_for_igvn(init_out);
> 
> I don't think a call to record_for_igvn() is needed here.

yes, record_for_igvn is called in gvn_replace_by.

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

PR: https://git.openjdk.java.net/jdk/pull/4238