Summary
Retroactive CSR for linked issue. Noticed late this affected a public field in the Vector API incubator.
The SUADD
field in src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorOperators.java
is being changed from Binary
to Associative
, a subtype of Binary
.
Problem
The SUADD
lane-wise operation is associative but is not type as such in VectorOperators
Solution
Updating the relevant flags on this public field and changing its type from Binary
to Associative
.
Specification
diff --git a/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorOperators.java b/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorOperators.java
index 0e231bd5174..f4da4f42934 100644
--- a/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorOperators.java
+++ b/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorOperators.java
@@ -577,7 +577,7 @@ static boolean opKind(Operator op, int bit) {
/** Produce saturating unsigned {@code a+b}. Integral only.
* @see VectorMath#addSaturatingUnsigned(int, int)
*/
- public static final Binary SUADD = binary("SUADD", "+", VectorSupport.VECTOR_OP_SUADD, VO_NOFP);
+ public static final Associative SUADD = assoc("SUADD", "+", VectorSupport.VECTOR_OP_SUADD, VO_NOFP+VO_ASSOC);
/** Produce saturating {@code a-b}. Integral only.
* @see VectorMath#subSaturating(int, int)
*/
- csr of
-
JDK-8358768 [vectorapi] Make VectorOperators.SUADD an Associative
-
- Resolved
-