RFR: 8358768: [vectorapi] Make VectorOperators.SUADD an Associative [v2]
Xiaohong Gong
xgong at openjdk.org
Fri Jul 11 03:50:39 UTC 2025
On Wed, 9 Jul 2025 22:52:58 GMT, Ian Graves <igraves at openjdk.org> wrote:
>> Adding SUADD an associative operation in the Vector API. Saturated addition on fixed-width unsigned integers is provably associative.
>
> Ian Graves has updated the pull request incrementally with one additional commit since the last revision:
>
> Adding masked associative tests
src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorOperators.java line 580:
> 578: * @see VectorMath#addSaturatingUnsigned(int, int)
> 579: */
> 580: public static final Associative SUADD = assoc("SUADD", "+", VectorSupport.VECTOR_OP_SUADD, VO_NOFP+VO_ASSOC);
Suggestion:
public static final Associative SUADD = assoc("SUADD", "+", VectorSupport.VECTOR_OP_SUADD, VO_NOFP + VO_ASSOC);
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26099#discussion_r2199446939
More information about the core-libs-dev
mailing list