RFR: 8347406: [REDO] C1/C2 don't handle allocation failure properly during initialization (RuntimeStub::new_runtime_stub fatal crash) [v4]

[Damon Fenacci]

On Thu, 20 Feb 2025 21:36:44 GMT, Dean Long <dlong at openjdk.org> wrote:

>> Damon Fenacci has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   JDK-8347406: re-add modified assert
>
> src/hotspot/share/opto/output.cpp line 3487:
> 
>> 3485:         C->record_failure("CodeCache is full");
>> 3486:       } else {
>> 3487:         C->set_stub_entry_point(rs->entry_point());
> 
> Is the deleted rs->is_runtime_stub() assert still useful here?

A slightly modified one surely is. Inserted it again.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23630#discussion_r1973446189