< prev index next >

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

Print this page
rev 58165 : Fix the issue that unslice modifies the argument vector
Summary: Change the name of getElements() to vec() and clone argument value to
avoid it being modified

*** 56,66 **** static final int VLENGTH = VSPECIES.laneCount(); static final Class<Short> ETYPE = short.class; // The JVM expects to find the state here. ! private final short[] vec; // Don't access directly, use getElements() instead. Short64Vector(short[] v) { vec = v; } --- 56,66 ---- static final int VLENGTH = VSPECIES.laneCount(); static final Class<Short> ETYPE = short.class; // The JVM expects to find the state here. ! private final short[] vec; // Don't access directly, use vec() instead. Short64Vector(short[] v) { vec = v; }
*** 117,127 **** public final int byteSize() { return VSIZE / Byte.SIZE; } /*package-private*/ @ForceInline final @Override ! short[] getElements() { return VectorIntrinsics.maybeRebox(this).vec; } // Virtualized constructors --- 117,127 ---- public final int byteSize() { return VSIZE / Byte.SIZE; } /*package-private*/ @ForceInline final @Override ! short[] vec() { return VectorIntrinsics.maybeRebox(this).vec; } // Virtualized constructors
*** 476,486 **** } return (short) VectorIntrinsics.extract( VCLASS, ETYPE, VLENGTH, this, i, (vec, ix) -> { ! short[] vecarr = vec.getElements(); return (long)vecarr[ix]; }); } @Override --- 476,486 ---- } return (short) VectorIntrinsics.extract( VCLASS, ETYPE, VLENGTH, this, i, (vec, ix) -> { ! short[] vecarr = vec.vec(); return (long)vecarr[ix]; }); } @Override
*** 490,500 **** } return VectorIntrinsics.insert( VCLASS, ETYPE, VLENGTH, this, i, (long)e, (v, ix, bits) -> { ! short[] res = v.getElements().clone(); res[ix] = (short)bits; return v.vectorFactory(res); }); } --- 490,500 ---- } return VectorIntrinsics.insert( VCLASS, ETYPE, VLENGTH, this, i, (long)e, (v, ix, bits) -> { ! short[] res = v.vec().clone(); res[ix] = (short)bits; return v.vectorFactory(res); }); }
< prev index next >