src/jdk.incubator.vector/share/classes/jdk/incubator/vector/Short256Vector.java

@@ -153,11 +153,11 @@
             throw new IllegalArgumentException("Vector length this species length differ");
 
         return VectorIntrinsics.cast(
             Short256Vector.class,
             short.class, LENGTH,
-            s.boxType(),
+            s.vectorType(),
             s.elementType(), LENGTH,
             this, s,
             (species, vector) -> vector.castDefault(species)
         );
     }

@@ -291,48 +291,48 @@
 
     @Override
     @ForceInline
     public ShortVector reshape(VectorSpecies<Short> s) {
         Objects.requireNonNull(s);
-        if (s.bitSize() == 64 && (s.boxType() == Short64Vector.class)) {
+        if (s.bitSize() == 64 && (s.vectorType() == Short64Vector.class)) {
             return VectorIntrinsics.reinterpret(
                 Short256Vector.class,
                 short.class, LENGTH,
                 Short64Vector.class,
                 short.class, Short64Vector.LENGTH,
                 this, s,
                 (species, vector) -> (ShortVector) vector.defaultReinterpret(species)
             );
-        } else if (s.bitSize() == 128 && (s.boxType() == Short128Vector.class)) {
+        } else if (s.bitSize() == 128 && (s.vectorType() == Short128Vector.class)) {
             return VectorIntrinsics.reinterpret(
                 Short256Vector.class,
                 short.class, LENGTH,
                 Short128Vector.class,
                 short.class, Short128Vector.LENGTH,
                 this, s,
                 (species, vector) -> (ShortVector) vector.defaultReinterpret(species)
             );
-        } else if (s.bitSize() == 256 && (s.boxType() == Short256Vector.class)) {
+        } else if (s.bitSize() == 256 && (s.vectorType() == Short256Vector.class)) {
             return VectorIntrinsics.reinterpret(
                 Short256Vector.class,
                 short.class, LENGTH,
                 Short256Vector.class,
                 short.class, Short256Vector.LENGTH,
                 this, s,
                 (species, vector) -> (ShortVector) vector.defaultReinterpret(species)
             );
-        } else if (s.bitSize() == 512 && (s.boxType() == Short512Vector.class)) {
+        } else if (s.bitSize() == 512 && (s.vectorType() == Short512Vector.class)) {
             return VectorIntrinsics.reinterpret(
                 Short256Vector.class,
                 short.class, LENGTH,
                 Short512Vector.class,
                 short.class, Short512Vector.LENGTH,
                 this, s,
                 (species, vector) -> (ShortVector) vector.defaultReinterpret(species)
             );
         } else if ((s.bitSize() > 0) && (s.bitSize() <= 2048)
-                && (s.bitSize() % 128 == 0) && (s.boxType() == ShortMaxVector.class)) {
+                && (s.bitSize() % 128 == 0) && (s.vectorType() == ShortMaxVector.class)) {
             return VectorIntrinsics.reinterpret(
                 Short256Vector.class,
                 short.class, LENGTH,
                 ShortMaxVector.class,
                 short.class, ShortMaxVector.LENGTH,

@@ -655,51 +655,78 @@
         return blend(xor(v), m);
     }
 
     @Override
     @ForceInline
-    public Short256Vector shiftL(int s) {
+    public Short256Vector shiftLeft(int s) {
         return VectorIntrinsics.broadcastInt(
             VECTOR_OP_LSHIFT, Short256Vector.class, short.class, LENGTH,
             this, s,
-            (v, i) -> v.uOp((__, a) -> (short) (a << (i & 15))));
+            (v, i) -> v.uOp((__, a) -> (short) (a << (i & 0xF))));
     }
 
     @Override
     @ForceInline
-    public Short256Vector shiftL(int s, VectorMask<Short> m) {
-        return blend(shiftL(s), m);
+    public Short256Vector shiftLeft(int s, VectorMask<Short> m) {
+        return blend(shiftLeft(s), m);
     }
 
     @Override
     @ForceInline
-    public Short256Vector shiftR(int s) {
+    public Short256Vector shiftLeft(Vector<Short> s) {
+        Short256Vector shiftv = (Short256Vector)s;
+        // As per shift specification for Java, mask the shift count.
+        shiftv = shiftv.and(ShortVector.broadcast(SPECIES, (short) 0xF));
+        return this.bOp(shiftv, (i, a, b) -> (short) (a << (b & 0xF)));
+    }
+
+    @Override
+    @ForceInline
+    public Short256Vector shiftRight(int s) {
         return VectorIntrinsics.broadcastInt(
             VECTOR_OP_URSHIFT, Short256Vector.class, short.class, LENGTH,
             this, s,
-            (v, i) -> v.uOp((__, a) -> (short) ((a & 0xFFFF) >>> (i & 15))));
+            (v, i) -> v.uOp((__, a) -> (short) ((a & 0xFFFF) >>> (i & 0xF))));
+    }
+
+    @Override
+    @ForceInline
+    public Short256Vector shiftRight(int s, VectorMask<Short> m) {
+        return blend(shiftRight(s), m);
     }
 
     @Override
     @ForceInline
-    public Short256Vector shiftR(int s, VectorMask<Short> m) {
-        return blend(shiftR(s), m);
+    public Short256Vector shiftRight(Vector<Short> s) {
+        Short256Vector shiftv = (Short256Vector)s;
+        // As per shift specification for Java, mask the shift count.
+        shiftv = shiftv.and(ShortVector.broadcast(SPECIES, (short) 0xF));
+        return this.bOp(shiftv, (i, a, b) -> (short) (a >>> (b & 0xF)));
     }
 
     @Override
     @ForceInline
-    public Short256Vector aShiftR(int s) {
+    public Short256Vector shiftArithmeticRight(int s) {
         return VectorIntrinsics.broadcastInt(
             VECTOR_OP_RSHIFT, Short256Vector.class, short.class, LENGTH,
             this, s,
-            (v, i) -> v.uOp((__, a) -> (short) (a >> (i & 15))));
+            (v, i) -> v.uOp((__, a) -> (short) (a >> (i & 0xF))));
+    }
+
+    @Override
+    @ForceInline
+    public Short256Vector shiftArithmeticRight(int s, VectorMask<Short> m) {
+        return blend(shiftArithmeticRight(s), m);
     }
 
     @Override
     @ForceInline
-    public Short256Vector aShiftR(int s, VectorMask<Short> m) {
-        return blend(aShiftR(s), m);
+    public Short256Vector shiftArithmeticRight(Vector<Short> s) {
+        Short256Vector shiftv = (Short256Vector)s;
+        // As per shift specification for Java, mask the shift count.
+        shiftv = shiftv.and(ShortVector.broadcast(SPECIES, (short) 0xF));
+        return this.bOp(shiftv, (i, a, b) -> (short) (a >> (b & 0xF)));
     }
     // Ternary operations
 
 
     // Type specific horizontal reductions

@@ -1027,21 +1054,21 @@
     }
 
 
 
     @Override
-    public Short256Vector rotateEL(int j) {
+    public Short256Vector rotateLanesLeft(int j) {
         short[] vec = getElements();
         short[] res = new short[length()];
         for (int i = 0; i < length(); i++){
             res[(j + i) % length()] = vec[i];
         }
         return new Short256Vector(res);
     }
 
     @Override
-    public Short256Vector rotateER(int j) {
+    public Short256Vector rotateLanesRight(int j) {
         short[] vec = getElements();
         short[] res = new short[length()];
         for (int i = 0; i < length(); i++){
             int z = i - j;
             if(j < 0) {

@@ -1052,21 +1079,21 @@
         }
         return new Short256Vector(res);
     }
 
     @Override
-    public Short256Vector shiftEL(int j) {
+    public Short256Vector shiftLanesLeft(int j) {
         short[] vec = getElements();
         short[] res = new short[length()];
         for (int i = 0; i < length() - j; i++) {
             res[i] = vec[i + j];
         }
         return new Short256Vector(res);
     }
 
     @Override
-    public Short256Vector shiftER(int j) {
+    public Short256Vector shiftLanesRight(int j) {
         short[] vec = getElements();
         short[] res = new short[length()];
         for (int i = 0; i < length() - j; i++){
             res[i + j] = vec[i];
         }