RFR: 8316702: Only evaluate buffer when IGVPrintLevelOption >= 5
Christian Hagedorn
chagedorn at openjdk.org
Fri Sep 22 06:33:10 UTC 2023
On Fri, 22 Sep 2023 01:53:24 GMT, Xin Liu <xliu at openjdk.org> wrote:
> This is a follow-up of JDK-8310220. hotspot C2 only emit IR per bytecode to IGV when level = 5.
>
> Currently, we conservatively check C->should_print_igv(1) in Parse::do_one_bytecode(). It's wasteful when level>=1 and <= 4. We evaluate `jio_snprintf(buffer, sizeof(buffer), "Bytecode %d: %s", bci(), Bytecodes::name(bc()))` for each bytecode but emit nothing.
>
> We should use the same level(5) in those 2 places.
Good catch, looks good!
-------------
Marked as reviewed by chagedorn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/15878#pullrequestreview-1639231955
More information about the hotspot-compiler-dev
mailing list