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