RFR: 8315916: assert(C->live_nodes() <= C->max_node_limit()) failed: Live Node limit exceeded [v9]

[Dhamoder Nalla]

On Wed, 9 Apr 2025 05:39:51 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Dhamoder Nalla has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Modify jtreg test
>
> test/hotspot/jtreg/compiler/escapeAnalysis/TestScalarizeBailout.java line 28:
> 
>> 26:  * @bug 8315916
>> 27:  * @summary Test early bailout during the creation of graph nodes for the scalarization of array fields, rather than during code generation.
>> 28:  * @run main/othervm/timeout=240000
> 
> That's a huge timeout. How long does this test need on your machine? If too long, can you also trigger the issue with a smaller `EliminateAllocationArraySizeLimit` than currently used?

updated the EliminateAllocationArraySizeLimit to 32k by reducing the array size.

> test/hotspot/jtreg/compiler/escapeAnalysis/TestScalarizeBailout.java line 29:
> 
>> 27:  * @summary Test early bailout during the creation of graph nodes for the scalarization of array fields, rather than during code generation.
>> 28:  * @run main/othervm/timeout=240000
>> 29:  *                   -Xcomp
> 
> Can you limit the performed compilations to a single or few methods with `compileonly` to trigger the assert? This will reduce the required time to run this test with `Xcomp`.

added compileonly options

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20504#discussion_r2035885049
PR Review Comment: https://git.openjdk.org/jdk/pull/20504#discussion_r2035886103