On Mon, 11 Oct 2021 09:55:28 GMT, Evgeny Nikitin <eniki...@openjdk.org> wrote:
> This PR contains a relatively simple test which verifies that JVMTI-agents > are correctly informed about exceptions caught in C2-compiled code. The > 8269574 introduces pre-allocated exceptions in some paths, so the test tries > to produce a number of various exceptions and check that provided small JVMTI > agent got notified about all of them. Hi Evgeny, New test looks good to me. I've inlined a couple of minor comments/suggestions. Thanks, Serguei test/hotspot/jtreg/compiler/jvmti/libTriggerBuiltinExceptions.cpp line 77: > 75: } > 76: > 77: } while (false); I'm not sure why the while (false) loop is needed. You can always return JNI_ERR instead of break in all places where the result != JVMTI_ERROR_NONE is detected and return JNI_OK at the end. Is it to for one-return style? ------------- Marked as reviewed by sspitsyn (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/5889