< prev index next >

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