RFR: 8338021: Support new unsigned and saturating vector operators in VectorAPI [v7]
Paul Sandoz
psandoz at openjdk.org
Fri Sep 6 18:02:12 UTC 2024
On Fri, 6 Sep 2024 06:40:18 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:
>> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorMathUtils.java line 78:
>>
>>> 76: * @since 24
>>> 77: */
>>> 78: public static long addSaturating(long a, long b) {
>>
>> Are these public methods any Java dev could use? If so: do we have tests for them?
>
> Made them package private. These routines are exercised by newly added jtreg tests.
These methods need to be public, as the need to be used in any tail computation. Recommend naming as `VectorMath` aligning with the naming of `Math` and `StrictMath`.
* The class {@code VectorMath} contains methods for performing
* scalar numeric operations in support of vector numeric operations.
For each method we can reference the associated vector operator.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20507#discussion_r1747520954
More information about the core-libs-dev
mailing list