From 6c5e8cc6cfe3d2981e2e842e09d63fb9e939df92 Mon Sep 17 00:00:00 2001 From: Jatin Bhateja Date: Fri, 27 Mar 2026 09:15:56 +0000 Subject: [PATCH] Review comments resolutions --- .../jtreg/compiler/vectorapi/TestMultiplyReductionByte.java | 6 ++++-- test/jdk/jdk/incubator/vector/ByteVector128Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ByteVector256Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ByteVector512Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ByteVector64Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ByteVectorMaxTests.java | 4 ++++ test/jdk/jdk/incubator/vector/DoubleVector128Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/DoubleVector256Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/DoubleVector512Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/DoubleVector64Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/DoubleVectorMaxTests.java | 4 ++++ test/jdk/jdk/incubator/vector/FloatVector128Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/FloatVector256Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/FloatVector512Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/FloatVector64Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/FloatVectorMaxTests.java | 4 ++++ test/jdk/jdk/incubator/vector/IntVector128Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/IntVector256Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/IntVector512Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/IntVector64Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/IntVectorMaxTests.java | 4 ++++ test/jdk/jdk/incubator/vector/LongVector128Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/LongVector256Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/LongVector512Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/LongVector64Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/LongVectorMaxTests.java | 4 ++++ test/jdk/jdk/incubator/vector/ShortVector128Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ShortVector256Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ShortVector512Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ShortVector64Tests.java | 4 ++++ test/jdk/jdk/incubator/vector/ShortVectorMaxTests.java | 4 ++++ .../jdk/jdk/incubator/vector/templates/Unit-header.template | 4 ++++ 32 files changed, 128 insertions(+), 2 deletions(-) diff --git a/test/hotspot/jtreg/compiler/vectorapi/TestMultiplyReductionByte.java b/test/hotspot/jtreg/compiler/vectorapi/TestMultiplyReductionByte.java index 87c9a8f8554..13eafd8dc26 100644 --- a/test/hotspot/jtreg/compiler/vectorapi/TestMultiplyReductionByte.java +++ b/test/hotspot/jtreg/compiler/vectorapi/TestMultiplyReductionByte.java @@ -31,6 +31,7 @@ import jdk.incubator.vector.ByteVector; import jdk.incubator.vector.VectorOperators; import jdk.test.lib.Asserts; +import jdk.test.lib.Utils; /** * @test @@ -41,16 +42,17 @@ import jdk.test.lib.Asserts; * when most lanes are 1 and a single lane differs. * @library /test/lib / * @modules jdk.incubator.vector - * @run driver compiler.vectorapi.TestMultiplyReductionByte + * @run driver ${test.main.class} */ public class TestMultiplyReductionByte { static byte[] input = new byte[64]; - static int pos; + static int pos = Utils.getRandomInstance().nextInt(input.length); static { Arrays.fill(input, (byte) 1); + input[pos] = -3; } @Test diff --git a/test/jdk/jdk/incubator/vector/ByteVector128Tests.java b/test/jdk/jdk/incubator/vector/ByteVector128Tests.java index ca6fa537ac4..b605182e496 100644 --- a/test/jdk/jdk/incubator/vector/ByteVector128Tests.java +++ b/test/jdk/jdk/incubator/vector/ByteVector128Tests.java @@ -1114,6 +1114,10 @@ public class ByteVector128Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((byte)(i + 1) == 0) ? 1 : (byte)(i + 1))); }), + withToString("byte[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (byte)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("byte[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ByteVector256Tests.java b/test/jdk/jdk/incubator/vector/ByteVector256Tests.java index 5c32d4a7f74..bc05a0bd6ca 100644 --- a/test/jdk/jdk/incubator/vector/ByteVector256Tests.java +++ b/test/jdk/jdk/incubator/vector/ByteVector256Tests.java @@ -1114,6 +1114,10 @@ public class ByteVector256Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((byte)(i + 1) == 0) ? 1 : (byte)(i + 1))); }), + withToString("byte[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (byte)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("byte[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ByteVector512Tests.java b/test/jdk/jdk/incubator/vector/ByteVector512Tests.java index 094f3bbebdc..45038c53824 100644 --- a/test/jdk/jdk/incubator/vector/ByteVector512Tests.java +++ b/test/jdk/jdk/incubator/vector/ByteVector512Tests.java @@ -1114,6 +1114,10 @@ public class ByteVector512Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((byte)(i + 1) == 0) ? 1 : (byte)(i + 1))); }), + withToString("byte[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (byte)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("byte[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ByteVector64Tests.java b/test/jdk/jdk/incubator/vector/ByteVector64Tests.java index e8ff81678cd..b350d0a8ee0 100644 --- a/test/jdk/jdk/incubator/vector/ByteVector64Tests.java +++ b/test/jdk/jdk/incubator/vector/ByteVector64Tests.java @@ -1114,6 +1114,10 @@ public class ByteVector64Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((byte)(i + 1) == 0) ? 1 : (byte)(i + 1))); }), + withToString("byte[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (byte)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("byte[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ByteVectorMaxTests.java b/test/jdk/jdk/incubator/vector/ByteVectorMaxTests.java index c53710c3fdd..ba2307f15b1 100644 --- a/test/jdk/jdk/incubator/vector/ByteVectorMaxTests.java +++ b/test/jdk/jdk/incubator/vector/ByteVectorMaxTests.java @@ -1120,6 +1120,10 @@ public class ByteVectorMaxTests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((byte)(i + 1) == 0) ? 1 : (byte)(i + 1))); }), + withToString("byte[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (byte)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("byte[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/DoubleVector128Tests.java b/test/jdk/jdk/incubator/vector/DoubleVector128Tests.java index 1457e5a51ca..7bec91b698b 100644 --- a/test/jdk/jdk/incubator/vector/DoubleVector128Tests.java +++ b/test/jdk/jdk/incubator/vector/DoubleVector128Tests.java @@ -1239,6 +1239,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((double)(i + 1) == 0) ? 1 : (double)(i + 1))); }), + withToString("double[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (double)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("double[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (double)0.01 + ((double)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/DoubleVector256Tests.java b/test/jdk/jdk/incubator/vector/DoubleVector256Tests.java index e417abe52e6..3da554e67e5 100644 --- a/test/jdk/jdk/incubator/vector/DoubleVector256Tests.java +++ b/test/jdk/jdk/incubator/vector/DoubleVector256Tests.java @@ -1239,6 +1239,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((double)(i + 1) == 0) ? 1 : (double)(i + 1))); }), + withToString("double[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (double)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("double[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (double)0.01 + ((double)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/DoubleVector512Tests.java b/test/jdk/jdk/incubator/vector/DoubleVector512Tests.java index d23b2bf1511..6c060714da8 100644 --- a/test/jdk/jdk/incubator/vector/DoubleVector512Tests.java +++ b/test/jdk/jdk/incubator/vector/DoubleVector512Tests.java @@ -1239,6 +1239,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((double)(i + 1) == 0) ? 1 : (double)(i + 1))); }), + withToString("double[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (double)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("double[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (double)0.01 + ((double)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/DoubleVector64Tests.java b/test/jdk/jdk/incubator/vector/DoubleVector64Tests.java index d2aff0b7ea5..9e34cbc3a31 100644 --- a/test/jdk/jdk/incubator/vector/DoubleVector64Tests.java +++ b/test/jdk/jdk/incubator/vector/DoubleVector64Tests.java @@ -1239,6 +1239,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((double)(i + 1) == 0) ? 1 : (double)(i + 1))); }), + withToString("double[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (double)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("double[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (double)0.01 + ((double)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/DoubleVectorMaxTests.java b/test/jdk/jdk/incubator/vector/DoubleVectorMaxTests.java index 2684ae2d0c6..74be9b42a8b 100644 --- a/test/jdk/jdk/incubator/vector/DoubleVectorMaxTests.java +++ b/test/jdk/jdk/incubator/vector/DoubleVectorMaxTests.java @@ -1245,6 +1245,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((double)(i + 1) == 0) ? 1 : (double)(i + 1))); }), + withToString("double[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (double)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("double[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (double)0.01 + ((double)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/FloatVector128Tests.java b/test/jdk/jdk/incubator/vector/FloatVector128Tests.java index 151ea17a886..eb4b686d41f 100644 --- a/test/jdk/jdk/incubator/vector/FloatVector128Tests.java +++ b/test/jdk/jdk/incubator/vector/FloatVector128Tests.java @@ -1256,6 +1256,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((float)(i + 1) == 0) ? 1 : (float)(i + 1))); }), + withToString("float[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (float)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("float[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (float)0.01 + ((float)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/FloatVector256Tests.java b/test/jdk/jdk/incubator/vector/FloatVector256Tests.java index 5315b69a5b6..f00bf19d383 100644 --- a/test/jdk/jdk/incubator/vector/FloatVector256Tests.java +++ b/test/jdk/jdk/incubator/vector/FloatVector256Tests.java @@ -1256,6 +1256,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((float)(i + 1) == 0) ? 1 : (float)(i + 1))); }), + withToString("float[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (float)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("float[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (float)0.01 + ((float)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/FloatVector512Tests.java b/test/jdk/jdk/incubator/vector/FloatVector512Tests.java index 6a958511439..573bcc59c63 100644 --- a/test/jdk/jdk/incubator/vector/FloatVector512Tests.java +++ b/test/jdk/jdk/incubator/vector/FloatVector512Tests.java @@ -1256,6 +1256,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((float)(i + 1) == 0) ? 1 : (float)(i + 1))); }), + withToString("float[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (float)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("float[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (float)0.01 + ((float)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/FloatVector64Tests.java b/test/jdk/jdk/incubator/vector/FloatVector64Tests.java index b06dff18194..8244ffabb51 100644 --- a/test/jdk/jdk/incubator/vector/FloatVector64Tests.java +++ b/test/jdk/jdk/incubator/vector/FloatVector64Tests.java @@ -1256,6 +1256,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((float)(i + 1) == 0) ? 1 : (float)(i + 1))); }), + withToString("float[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (float)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("float[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (float)0.01 + ((float)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/FloatVectorMaxTests.java b/test/jdk/jdk/incubator/vector/FloatVectorMaxTests.java index 9b84e852c1c..30c470e05f4 100644 --- a/test/jdk/jdk/incubator/vector/FloatVectorMaxTests.java +++ b/test/jdk/jdk/incubator/vector/FloatVectorMaxTests.java @@ -1262,6 +1262,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> (((float)(i + 1) == 0) ? 1 : (float)(i + 1))); }), + withToString("float[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (float)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("float[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS, i -> (float)0.01 + ((float)i / (i + 1))); diff --git a/test/jdk/jdk/incubator/vector/IntVector128Tests.java b/test/jdk/jdk/incubator/vector/IntVector128Tests.java index d62f5d8df00..2b36a2802eb 100644 --- a/test/jdk/jdk/incubator/vector/IntVector128Tests.java +++ b/test/jdk/jdk/incubator/vector/IntVector128Tests.java @@ -1104,6 +1104,10 @@ public class IntVector128Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((int)(i + 1) == 0) ? 1 : (int)(i + 1))); }), + withToString("int[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (int)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("int[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/IntVector256Tests.java b/test/jdk/jdk/incubator/vector/IntVector256Tests.java index bb2d1d717f4..9d4632b88dd 100644 --- a/test/jdk/jdk/incubator/vector/IntVector256Tests.java +++ b/test/jdk/jdk/incubator/vector/IntVector256Tests.java @@ -1104,6 +1104,10 @@ public class IntVector256Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((int)(i + 1) == 0) ? 1 : (int)(i + 1))); }), + withToString("int[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (int)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("int[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/IntVector512Tests.java b/test/jdk/jdk/incubator/vector/IntVector512Tests.java index 5ceba4e88ec..aa3c3eccf5b 100644 --- a/test/jdk/jdk/incubator/vector/IntVector512Tests.java +++ b/test/jdk/jdk/incubator/vector/IntVector512Tests.java @@ -1104,6 +1104,10 @@ public class IntVector512Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((int)(i + 1) == 0) ? 1 : (int)(i + 1))); }), + withToString("int[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (int)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("int[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/IntVector64Tests.java b/test/jdk/jdk/incubator/vector/IntVector64Tests.java index 9d3849a2c04..89247f9f932 100644 --- a/test/jdk/jdk/incubator/vector/IntVector64Tests.java +++ b/test/jdk/jdk/incubator/vector/IntVector64Tests.java @@ -1104,6 +1104,10 @@ public class IntVector64Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((int)(i + 1) == 0) ? 1 : (int)(i + 1))); }), + withToString("int[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (int)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("int[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/IntVectorMaxTests.java b/test/jdk/jdk/incubator/vector/IntVectorMaxTests.java index 6c671a81bf1..3272dbbb3de 100644 --- a/test/jdk/jdk/incubator/vector/IntVectorMaxTests.java +++ b/test/jdk/jdk/incubator/vector/IntVectorMaxTests.java @@ -1110,6 +1110,10 @@ public class IntVectorMaxTests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((int)(i + 1) == 0) ? 1 : (int)(i + 1))); }), + withToString("int[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (int)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("int[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/LongVector128Tests.java b/test/jdk/jdk/incubator/vector/LongVector128Tests.java index 49e2e1a0078..30f1ecc2bf4 100644 --- a/test/jdk/jdk/incubator/vector/LongVector128Tests.java +++ b/test/jdk/jdk/incubator/vector/LongVector128Tests.java @@ -1088,6 +1088,10 @@ public class LongVector128Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((long)(i + 1) == 0) ? 1 : (long)(i + 1))); }), + withToString("long[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (long)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("long[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/LongVector256Tests.java b/test/jdk/jdk/incubator/vector/LongVector256Tests.java index 84c8da4e800..6dd2f359742 100644 --- a/test/jdk/jdk/incubator/vector/LongVector256Tests.java +++ b/test/jdk/jdk/incubator/vector/LongVector256Tests.java @@ -1088,6 +1088,10 @@ public class LongVector256Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((long)(i + 1) == 0) ? 1 : (long)(i + 1))); }), + withToString("long[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (long)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("long[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/LongVector512Tests.java b/test/jdk/jdk/incubator/vector/LongVector512Tests.java index 00d8c4b010a..6347a1b53a2 100644 --- a/test/jdk/jdk/incubator/vector/LongVector512Tests.java +++ b/test/jdk/jdk/incubator/vector/LongVector512Tests.java @@ -1088,6 +1088,10 @@ public class LongVector512Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((long)(i + 1) == 0) ? 1 : (long)(i + 1))); }), + withToString("long[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (long)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("long[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/LongVector64Tests.java b/test/jdk/jdk/incubator/vector/LongVector64Tests.java index d4cdfe54ad2..0518c17b75f 100644 --- a/test/jdk/jdk/incubator/vector/LongVector64Tests.java +++ b/test/jdk/jdk/incubator/vector/LongVector64Tests.java @@ -1088,6 +1088,10 @@ public class LongVector64Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((long)(i + 1) == 0) ? 1 : (long)(i + 1))); }), + withToString("long[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (long)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("long[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/LongVectorMaxTests.java b/test/jdk/jdk/incubator/vector/LongVectorMaxTests.java index 597169b00f5..f5ab956b5a7 100644 --- a/test/jdk/jdk/incubator/vector/LongVectorMaxTests.java +++ b/test/jdk/jdk/incubator/vector/LongVectorMaxTests.java @@ -1094,6 +1094,10 @@ public class LongVectorMaxTests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((long)(i + 1) == 0) ? 1 : (long)(i + 1))); }), + withToString("long[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (long)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("long[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ShortVector128Tests.java b/test/jdk/jdk/incubator/vector/ShortVector128Tests.java index 85e7d715182..40de51c90ab 100644 --- a/test/jdk/jdk/incubator/vector/ShortVector128Tests.java +++ b/test/jdk/jdk/incubator/vector/ShortVector128Tests.java @@ -1104,6 +1104,10 @@ public class ShortVector128Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((short)(i + 1) == 0) ? 1 : (short)(i + 1))); }), + withToString("short[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (short)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("short[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ShortVector256Tests.java b/test/jdk/jdk/incubator/vector/ShortVector256Tests.java index df379b64e3a..c28dd34c8ef 100644 --- a/test/jdk/jdk/incubator/vector/ShortVector256Tests.java +++ b/test/jdk/jdk/incubator/vector/ShortVector256Tests.java @@ -1104,6 +1104,10 @@ public class ShortVector256Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((short)(i + 1) == 0) ? 1 : (short)(i + 1))); }), + withToString("short[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (short)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("short[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ShortVector512Tests.java b/test/jdk/jdk/incubator/vector/ShortVector512Tests.java index ae5804c3688..703ef7d358b 100644 --- a/test/jdk/jdk/incubator/vector/ShortVector512Tests.java +++ b/test/jdk/jdk/incubator/vector/ShortVector512Tests.java @@ -1104,6 +1104,10 @@ public class ShortVector512Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((short)(i + 1) == 0) ? 1 : (short)(i + 1))); }), + withToString("short[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (short)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("short[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ShortVector64Tests.java b/test/jdk/jdk/incubator/vector/ShortVector64Tests.java index 5f84682f02f..ea6846498a3 100644 --- a/test/jdk/jdk/incubator/vector/ShortVector64Tests.java +++ b/test/jdk/jdk/incubator/vector/ShortVector64Tests.java @@ -1104,6 +1104,10 @@ public class ShortVector64Tests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((short)(i + 1) == 0) ? 1 : (short)(i + 1))); }), + withToString("short[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (short)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("short[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/ShortVectorMaxTests.java b/test/jdk/jdk/incubator/vector/ShortVectorMaxTests.java index 5451ad6c50e..6a85da12054 100644 --- a/test/jdk/jdk/incubator/vector/ShortVectorMaxTests.java +++ b/test/jdk/jdk/incubator/vector/ShortVectorMaxTests.java @@ -1110,6 +1110,10 @@ public class ShortVectorMaxTests extends AbstractVectorTest { return fill(s * BUFFER_REPS, i -> (((short)(i + 1) == 0) ? 1 : (short)(i + 1))); }), + withToString("short[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> (short)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), withToString("short[cornerCaseValue(i)]", (int s) -> { return fill(s * BUFFER_REPS, i -> cornerCaseValue(i)); diff --git a/test/jdk/jdk/incubator/vector/templates/Unit-header.template b/test/jdk/jdk/incubator/vector/templates/Unit-header.template index e1434dccb2b..b87012a2181 100644 --- a/test/jdk/jdk/incubator/vector/templates/Unit-header.template +++ b/test/jdk/jdk/incubator/vector/templates/Unit-header.template @@ -1375,6 +1375,10 @@ relativeError)); return fill(s * BUFFER_REPS, i -> ((($type$)(i + 1) == 0) ? 1 : ($type$)(i + 1))); }), + withToString("$type$[smallOddValue(i)]", (int s) -> { + return fill(s * BUFFER_REPS, + i -> ($type$)(i % 7 == 0 ? -3 : (i % 3 == 0 ? -1 : 1))); + }), #if[FP] withToString("$type$[0.01 + (i / (i + 1))]", (int s) -> { return fill(s * BUFFER_REPS,