Change IRNode to use JDK-8310308 features

chhagedorn · Nov 8, 2023 · 8c404ce · 8c404ce
1 parent 37a344e
commit 8c404ce
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/test/hotspot/jtreg/compiler/lib/ir_framework/IRNode.java b/test/hotspot/jtreg/compiler/lib/ir_framework/IRNode.java
@@ -932,9 +932,9 @@ public class IRNode {
         beforeMatchingNameRegex(MUL_ADD_S2I, "MulAddS2I");
     }
 
-    public static final String MUL_ADD_VS2VI = PREFIX + "MUL_ADD_VS2VI" + POSTFIX;
+    public static final String MUL_ADD_VS2VI = VECTOR_PREFIX + "MUL_ADD_VS2VI" + POSTFIX;
     static {
-        superWordNodes(MUL_ADD_VS2VI, "MulAddVS2VI");
+        vectorNode(MUL_ADD_VS2VI, "MulAddVS2VI", TYPE_INT);
     }
 
     // Can only be used if avx512_vnni is available.

diff --git a/test/hotspot/jtreg/compiler/loopopts/superword/TestMulAddS2I.java b/test/hotspot/jtreg/compiler/loopopts/superword/TestMulAddS2I.java
@@ -46,8 +46,8 @@ public class TestMulAddS2I {
 
     static {
         for (int i = 0; i < RANGE; i++) {
-            sArr1[i] = (short)(i + (1000 * AbstractInfo.getRandom().nextDouble()));
-            sArr2[i] = (short)(RANGE - i + (2000 * AbstractInfo.getRandom().nextDouble()));
+            sArr1[i] = (short)(AbstractInfo.getRandom().nextInt());
+            sArr2[i] = (short)(AbstractInfo.getRandom().nextInt());
         }
         GOLDEN = test();
     }
@@ -80,7 +80,7 @@ public static void compare(int[] out) {
     @IR(applyIfCPUFeature = {"sse2", "true"}, applyIf = {"UseUnalignedLoadStores", "false"},
         failOn = {IRNode.MUL_ADD_VS2VI}, // Can only pack LoadS if UseUnalignedLoadStores is true (default if sse4.2)
         counts = {IRNode.MUL_ADD_S2I, "> 0"})
-    @IR(applyIfCPUFeature = {"asimd", "true"}, applyIf = {"MaxVectorSize", "16"},
+    @IR(applyIfCPUFeature = {"asimd", "true"}, applyIf = {"MaxVectorSize", "16"}, // AD file requires vector_length = 16
             counts = {IRNode.MUL_ADD_S2I, "> 0", IRNode.MUL_ADD_VS2VI, "> 0"})
     @IR(applyIfCPUFeature = {"avx512_vnni", "true"}, applyIf = {"UseUnalignedLoadStores", "true"},
             counts = {IRNode.MUL_ADD_S2I, "> 0", IRNode.MUL_ADD_VS2VI_VNNI, "> 0"})