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

[Dean Long]

On Thu, 27 Feb 2025 12:03:16 GMT, Damon Fenacci <dfenacci at openjdk.org> wrote:

>> 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.

I was thinking it could be moved into the `else` clause and simplified further.

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

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