On Wed, 9 Jul 2025 22:52:58 GMT, Ian Graves <igra...@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
