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