RFR: 8360641: TestCompilerCounts fails after 8354727 [v3]

Andrey Turbanov aturbanov at openjdk.org
Mon Jun 30 19:51:49 UTC 2025


On Mon, 30 Jun 2025 08:58:07 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:

>> After integrating #25872 the calculation of the`CICompilerCount` ergonomic became dependent on the size of `NonNMethodCodeHeapSize`, which itself is an ergonomic based on the available memory. Thus, depending on the system, the test `compiler/arguments/TestCompilerCounts.java` failed, i.e. locally this failed, but not on CI servers.
>> 
>> This PR changes the test to reflect the changes introduced in #25872.
>> 
>> Testing:
>>  - [ ] [Github Actions](https://github.com/mhaessig/jdk/actions/runs/15932906313)
>>  - [x] tier1,tier2 plus Oracle internal testing
>
> Manuel Hässig has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Remove superfluous newline
>  - Add copyright

test/hotspot/jtreg/compiler/arguments/TestCompilerCounts.java line 159:

> 157:             // Tiered modes
> 158:             int tieredCount = heuristicCount(cpus, Compilation.Tiered, debug);
> 159:             pass(tieredCount, opt, "-XX:NonNMethodCodeHeapSize="  +  NonNMethodCodeHeapSize);

Suggestion:

            pass(tieredCount, opt, "-XX:NonNMethodCodeHeapSize=" + NonNMethodCodeHeapSize);

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26024#discussion_r2175779955


More information about the hotspot-compiler-dev mailing list