Integrated: 8292395: [testbug] VectorGatherScatterTest.java fails on SVE with -XX:MaxVectorSize=8 after JDK-8288397

[Hao Sun]

On Tue, 16 Aug 2022 07:40:12 GMT, Hao Sun <haosun at openjdk.org> wrote:

> This test case was introduced in JDK-8288397 to check the misused
> register issue for gather load/scatter store rules. `I_SPECIES` was set
> intentionally as half of the length of `L_SPECIES` so as to trigger the
> register issue.
> 
> However, if "-XX:MaxVectorSize=8" is specified from command line,
> invalid vector size, i.e. 32-bit, is set for IntVector, leading to the
> clinit failure, while initializing `I_SPECIES`.
> 
> To bypass this failure, we check "MaxVectorSize" option at the
> "requires" annotation part in this patch, that is, we limit the minimal
> value of "MaxVectorSize" as 16 bytes.
> 
> Testing:
> 
> 1. We ran the test case on 512-bit SVE machine with 1) the default
> MaxVectorSize, 2) MaxVectorSize=8 and 3) MaxVectorSize=16, and the test
> passed. Note that in 2), this test case is actullay not selected since
> the condition at the "requires" annotation is not satisfied.
> 
> 2. We also manually checked that this patch doesn't change the
> functionality of this test case, i.e. checking the register issue.

This pull request has now been integrated.

Changeset: f75da223
Author:    Hao Sun <haosun at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/f75da2235ab7e33927729fa060ec4d86fdb0240f
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8292395: [testbug] VectorGatherScatterTest.java fails on SVE with -XX:MaxVectorSize=8 after JDK-8288397

Reviewed-by: chagedorn, thartmann

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

PR: https://git.openjdk.org/jdk/pull/9888