RFR: 8258751: Improve ExceptionHandlerTable dump

Tobias Hartmann thartmann at openjdk.java.net
Mon Jan 4 07:37:54 UTC 2021


On Wed, 23 Dec 2020 10:16:29 GMT, Hao Sun <github.com+16932759+shqking at openjdk.org> wrote:

> In addition to PC offset, print the real PC as well if available when
> dumping the ExceptionHandlerTable, as it's more convenient and readable
> for debugging or other activities.
> 
> Note that compilers might dump the ExceptionHandlerTable before method
> registered. See functions emit_code_body() and fill_buffer(). However,
> real PC is not ready yet during these phases since the generated code is
> not installed. In such scenarios, the real PC is not printed.
> 
> The example below shows how the dump information would be changed after
> applying this patch.
>   BEFORE:
>     ExceptionHandlerTable (size = 40 bytes)
>     catch_pco = 392 (2 entries)
>       bci 8 at scope depth 0 -> pco 184
>       bci 19 at scope depth 0 -> pco 124
>   AFTER:
>     ExceptionHandlerTable (size = 40 bytes)
>     catch_pco = 392 (pc=0x0000ffff818f13c8, 2 entries)
>       bci 8 at scope depth 0 -> pco 184 (pc=0x0000ffff818f12f8)
>       bci 19 at scope depth 0 -> pco 124 (pc=0x0000ffff818f12bc)

Looks good to me.

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

Marked as reviewed by thartmann (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1877


More information about the hotspot-compiler-dev mailing list