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