RFR: 8257910: [JVMCI] Set exception_seen accordingly in the runtime. [v2]

Vladimir Kozlov kvn at openjdk.java.net
Thu Dec 10 00:15:37 UTC 2020


On Wed, 9 Dec 2020 23:06:47 GMT, Yudi Zheng <yzheng at openjdk.org> wrote:

>> Update exception_seen flag in the MDO when no exception handler is found for caller during deoptimization, or when exception is thrown in the runtime.
>> 
>> Plus, `-Xlog:exceptions=info` code and `AbortVMOnException` are adjusted in C1 runtime and jvmci runtime to be consistent with C2, i.e., before `JvmtiExport::can_post_on_exceptions` deoptimization and fast continuation when compiled handler is found.
>
> Yudi Zheng has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Address comments.

Changes requested by kvn (Reviewer).

src/hotspot/share/jvmci/jvmciRuntime.cpp line 282:

> 280:   }
> 281:   // for AbortVMOnException flag
> 282:   NOT_PRODUCT(Exceptions::debug_check_abort(exception));

All other places calls debug_check_abort() in product too. It seems 8136577 changes missed JVMCI:
https://github.com/openjdk/jdk/commit/a25ce80e782a73390ae1109250dfa6fe6a48fcbe
Please, update to call it in product for JVMCI too. Note, AbortVMOnException is diagnostic flag now.

src/hotspot/share/c1/c1_Runtime1.cpp line 547:

> 545:     stringStream tempst;
> 546:     assert(nm->method() != NULL, "Unexpected NULL method()");
> 547:     tempst.print("compiled method <%s>\n"

Can you change message to?:
"C1 compiled method <%s>\n"

src/hotspot/share/jvmci/jvmciRuntime.cpp line 276:

> 274:     stringStream tempst;
> 275:     assert(cm->method() != NULL, "Unexpected null method()");
> 276:     tempst.print("compiled method <%s>\n"

Can you change message to?:
"JVMCI compiled method <%s>\n"

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

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


More information about the hotspot-dev mailing list