RFR: 8348401: [IR Framework] PrintTimes should not require verbose

Vladimir Kozlov kvn at openjdk.org
Thu Jan 23 19:38:47 UTC 2025


On Thu, 23 Jan 2025 12:32:35 GMT, Theo Weidmann <tweidmann at openjdk.org> wrote:

> The flag -DPrintTimes=true currently only prints the measured execution time if -DVerbose=true is also used. This patch removes the dependency on -DVerbose=true. 
> 
> Example command:
> 
> 
> jtreg -jdk:build/fastdeb/jdk -verbose:all -vmoptions:"-DPrintTimes=true"
> 
> 
> Example output:
> 
> 
> Messages from Test VM
> ---------------------
> [PRINT_TIMES] testIntArrayRunner:             66530333 ns
> [PRINT_TIMES] testShortArrayRunner:          108031125 ns
> [PRINT_TIMES] testLongArrayRunner:           159549917 ns
> [PRINT_TIMES] testByteArrayRunner:           201181042 ns

Need more explanation why Verbose is required to print times.
`||` is used in condition. I would assume Verbose is not required.
How calling `TestFrameworkSocket.write()` helps?
What about first line "Test execution times:"?

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

PR Review: https://git.openjdk.org/jdk/pull/23267#pullrequestreview-2570849979


More information about the hotspot-compiler-dev mailing list