RFR: 8314020: Print instruction blocks in byte units

Aleksey Shipilev shade at openjdk.org
Fri Aug 11 18:20:29 UTC 2023


On Fri, 11 Aug 2023 17:30:47 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> Is there any reason to retain the "unitsize" argument in os::print_instructions? I don't find any remaining uses of it.

Good question! I went back and forth on this. First, I wanted to keep other callers happy, in case _some_ arch would still want to do larger than 1-byte unit. On the other hand, this is not a generic `print_hex_dump`, we might as well drop the unit size. On the yet another hand, it is a single-line change, so it does not improve readability all that much. So... keeping the default argument seems a safe bet?

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

PR Comment: https://git.openjdk.org/jdk/pull/15211#issuecomment-1675180653


More information about the hotspot-runtime-dev mailing list