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

[Dhamoder Nalla]

On Wed, 27 Nov 2024 08:38:53 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Dhamoder Nalla has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   CR comments
>
> test/hotspot/jtreg/compiler/escapeAnalysis/TestScalarizeBailout.java line 42:
> 
>> 40:     try {
>> 41:       // load the class to initialize the static object and trigger the EA
>> 42:       Class <?> Class37 = Class.forName("compiler.escapeAnalysis.TestScalarizeBailout");
> 
> I'm still unclear why we need this line. `TestScalarizeBailout` should already be loaded since you are compiling `main()` from it.  Is it possible to trigger the assert without `forName()` somehow?
> 
> I tried to run your JTreg test but could not trigger the assert. On what platform/setup could you trigger this?

Sorry for the late response on this PR. I have modified the test, as the earlier version was reproducible only with the Java binary, not with jtreg.
The current version should trigger an assertion without this fix when using the jtreg tool.

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

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