RFR: 8274982: Add a test for 8269574.

David Holmes dholmes at openjdk.java.net
Thu Nov 11 06:09:36 UTC 2021


On Mon, 11 Oct 2021 09:55:28 GMT, Evgeny Nikitin <enikitin at 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.

Just a couple of minor issues, not a review of functionality.

test/hotspot/jtreg/compiler/jvmti/TriggerBuiltinExceptionsTest.java line 28:

> 26:  * @bug 8269574
> 27:  * @summary Verifies that exceptions are reported correctly to JVMTI in the compiled code
> 28:  * @requires vm.jvmti

You also require the JIT

test/hotspot/jtreg/compiler/jvmti/TriggerBuiltinExceptionsTest.java line 59:

> 57: public class TriggerBuiltinExceptionsTest {
> 58:     private static final WhiteBox WB = WhiteBox.getWhiteBox();
> 59:     private static final int ITERATIONS = 30;           //Arbitrary value, feel free to change

Style nit: space after //

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

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


More information about the hotspot-compiler-dev mailing list