RFR (XS): 8222292: runtime/exceptionMsgs/ArrayIndexOutOfBoundsException/ArrayIndexOutOfBoundsExceptionTest.java timeout but test passed
Vladimir Kozlov
vladimir.kozlov at oracle.com
Thu May 9 16:58:57 UTC 2019
Looks good.
We may need to do additional @run for libgraal later but for now this is good.
Thanks,
Vladimir
On 5/8/19 11:50 PM, David Holmes wrote:
> Bug: https://bugs.openjdk.java.net/browse/JDK-8222292
> webrev: http://cr.openjdk.java.net/~dholmes/8222292/webrev/ (diff also below)
>
> The test is checking exception messages in three compilation modes using three @run directives:
> - "normal" (start interpreted)
> - C2 (via Xcomp 'no tiered')
> - C1 (via Xcomp 'stop at level 1')
>
> when the test is run with Graal enabled it doesn't make sense to run these three variants (i.e.
> Graal in C1/C2 mode is nonsensical - you get Graal with some flags that affect how Graal runs). It
> also turns out the "C1-Graal" variant takes 20+ minutes to execute so the test times out.
>
> Simple fix: split the three tests into two distinct @test definitions. The first runs "normal" with
> or without Graal. The second runs C1/C2 only when Graal is disabled. End result: we test four
> compiler variants, which is what we want. (Actually there is a fifth as we'll also test "normal"
> with Xcomp applied to it.)
>
> Thanks,
> David
> -----
>
> ---
> old/test/hotspot/jtreg/runtime/exceptionMsgs/ArrayIndexOutOfBoundsException/ArrayIndexOutOfBoundsExceptionTest.java
> 2019-05-09 02:34:49.740597326 -0400
> +++
> new/test/hotspot/jtreg/runtime/exceptionMsgs/ArrayIndexOutOfBoundsException/ArrayIndexOutOfBoundsExceptionTest.java
> 2019-05-09 02:34:49.267569730 -0400
> @@ -26,8 +26,14 @@
> * @test
> * @summary Test extended ArrayIndexOutOfBoundsException message. The
> * message lists information about the array and the indexes involved.
> - * @compile ArrayIndexOutOfBoundsExceptionTest.java
> + * @comment This will run in 'normal' mode when Graal is not enabled, else
> + * Graal mode.
> * @run testng ArrayIndexOutOfBoundsExceptionTest
> + */
> +/**
> + * @test
> + * @requires !vm.graal.enabled
> + * @comment These test C1 and C2 so make no sense when Graal is enabled.
> * @run testng/othervm -Xcomp -XX:-TieredCompilation ArrayIndexOutOfBoundsExceptionTest
> * @run testng/othervm -Xcomp -XX:TieredStopAtLevel=1 ArrayIndexOutOfBoundsExceptionTest
> */
More information about the hotspot-runtime-dev
mailing list