RFR: JDK-8318183: C2: VM may crash after hitting node limit

[Thomas Stuefe]

On Tue, 17 Oct 2023 18:20:34 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> @TobiHartmann I used GraphKit::failing() as you suggested. One problem I see is that this relies on the local variable C in the Phase super class, and that depends on the phasenumber handed in to the Phase() constructor, and it could theoretically be null. But since all other places that call `failing()` don't seem to care either, I don't think its an issue, right?
>
>> @TobiHartmann I used GraphKit::failing() as you suggested. One problem I see is that this relies on the local variable C in the Phase super class, and that depends on the phasenumber handed in to the Phase() constructor, and it could theoretically be null. But since all other places that call `failing()` don't seem to care either, I don't think its an issue, right?
> 
> It is only `nullptr` for Compile class where you don't need `C`.

Thank you @vnkozlov and @TobiHartmann !

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

PR Comment: https://git.openjdk.org/jdk/pull/16205#issuecomment-1767957619