test/hotspot/jtreg/compiler/vectorization/runner/ArrayTypeConvertTest.java

*** 59,11 ***
  
  import compiler.lib.ir_framework.*;
  
  // Explanation about AlignVector: we require 8-byte alignment of all addresses.
  // But the array base offset changes with UseCompactObjectHeaders.
! // This means it affects the alignment constraints.
  
  public class ArrayTypeConvertTest extends VectorizationTestRunner {
  
      // We must pass the flags directly to the test-VM, and not the driver vm in the @run above.
      @Override
--- 59,11 ---
  
  import compiler.lib.ir_framework.*;
  
  // Explanation about AlignVector: we require 8-byte alignment of all addresses.
  // But the array base offset changes with UseCompactObjectHeaders.
! // It should, however, not affect the alignment constraints.
  
  public class ArrayTypeConvertTest extends VectorizationTestRunner {
  
      // We must pass the flags directly to the test-VM, and not the driver vm in the @run above.
      @Override

*** 232,23 ***
      }
  
      // ---------------- Convert Subword-I to F/D ----------------
      @Test
      @IR(applyIfCPUFeatureOr = {"asimd", "true", "avx2", "true", "rvv", "true"},
-         applyIfOr = {"AlignVector", "false", "UseCompactObjectHeaders", "false"},
          counts = {IRNode.VECTOR_CAST_S2F, IRNode.VECTOR_SIZE + "min(max_short, max_float)", ">0"})
      public float[] convertShortToFloat() {
          float[] res = new float[SIZE];
          for (int i = 0; i < SIZE; i++) {
              res[i] = (float) shorts[i];
-             // AlignVector=true requires that all vector load/store are 8-byte aligned.
-             // F_adr = base + UNSAFE.ARRAY_FLOAT_BASE_OFFSET + 4*i
-             //                = 16 (UseCompactObjectHeaders=false)    -> i % 2 = 0
-             //                = 12 (UseCompactObjectHeaders=true )    -> i % 2 = 1
-             // S_adr = base + UNSAFE.ARRAY_SHORT_BASE_OFFSET + 2*i
-             //                = 16 (UseCompactObjectHeaders=false)    -> i % 4 = 0  -> can align both
-             //                = 12 (UseCompactObjectHeaders=true )    -> i % 4 = 2  -> cannot align both
          }
          return res;
      }
  
      @Test
--- 232,15 ---

*** 335,42 ***
      }
  
      // ---------------- Convert F/D to Subword-I ----------------
      @Test
      @IR(applyIfCPUFeatureOr = {"asimd", "true", "avx2", "true", "rvv", "true"},
-         applyIfOr = {"AlignVector", "false", "UseCompactObjectHeaders", "false"},
          counts = {IRNode.VECTOR_CAST_F2S, IRNode.VECTOR_SIZE + "min(max_float, max_short)", ">0"})
      public short[] convertFloatToShort() {
          short[] res = new short[SIZE];
          for (int i = 0; i < SIZE; i++) {
              res[i] = (short) floats[i];
-             // AlignVector=true requires that all vector load/store are 8-byte aligned.
-             // F_adr = base + UNSAFE.ARRAY_FLOAT_BASE_OFFSET + 4*i
-             //                = 16 (UseCompactObjectHeaders=false)    -> i % 2 = 0
-             //                = 12 (UseCompactObjectHeaders=true )    -> i % 2 = 1
-             // S_adr = base + UNSAFE.ARRAY_SHORT_BASE_OFFSET + 2*i
-             //                = 16 (UseCompactObjectHeaders=false)    -> i % 4 = 0  -> can align both
-             //                = 12 (UseCompactObjectHeaders=true )    -> i % 4 = 2  -> cannot align both
          }
          return res;
      }
  
      @Test
      @IR(applyIfCPUFeatureOr = {"asimd", "true", "avx2", "true", "rvv", "true"},
-         applyIfOr = {"AlignVector", "false", "UseCompactObjectHeaders", "false"},
          counts = {IRNode.VECTOR_CAST_F2S, IRNode.VECTOR_SIZE + "min(max_float, max_char)", ">0"})
      public char[] convertFloatToChar() {
          char[] res = new char[SIZE];
          for (int i = 0; i < SIZE; i++) {
              res[i] = (char) floats[i];
-             // AlignVector=true requires that all vector load/store are 8-byte aligned.
-             // F_adr = base + UNSAFE.ARRAY_FLOAT_BASE_OFFSET + 4*i
-             //                = 16 (UseCompactObjectHeaders=false)    -> i % 2 = 0
-             //                = 12 (UseCompactObjectHeaders=true )    -> i % 2 = 1
-             // S_adr = base + UNSAFE.ARRAY_SHORT_BASE_OFFSET + 2*i
-             //                = 16 (UseCompactObjectHeaders=false)    -> i % 4 = 0  -> can align both
-             //                = 12 (UseCompactObjectHeaders=true )    -> i % 4 = 2  -> cannot align both
          }
          return res;
      }
  
      @Test
--- 327,26 ---