RFR: 8344009: Improve compiler memory statistics [v6]
Thomas Stuefe
stuefe at openjdk.org
Sun Mar 2 07:04:55 UTC 2025
On Thu, 27 Feb 2025 10:11:37 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:
>> Thomas Stuefe has updated the pull request incrementally with five additional commits since the last revision:
>>
>> - feedback ashu
>> - feedback roberto
>> - final-statistics-switch
>> - performance fix
>> - remove test code
>
> src/hotspot/share/runtime/globals.hpp line 1402:
>
>> 1400: "Print metaspace statistics upon VM exit.") \
>> 1401: \
>> 1402: product(bool, PrintCompilerMemoryStatisticsAtExit, false, DIAGNOSTIC, \
>
> Would it be possible to add a test for this new flag, perhaps by extending the existing test logic in `CompileCommandPrintMemStat`?
The test is already there - we test the final print output in CompileCommandPrintMemStat.java. I will change the logic however and only print the final output if `PrintCompilerMemoryStatisticsAtExit` is given (before, it was also printed, and tested as part of, CompilerCommand memstat). Then I will explicitly pass this flag in the test. That should be good enough.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23530#discussion_r1976555741
More information about the hotspot-dev
mailing list