RFR: 8376186: [VectorAPI] Nomenclature change for concrete vector classes

Paul Sandoz psandoz at openjdk.org
Wed Feb 11 09:29:16 UTC 2026


On Fri, 6 Feb 2026 18:33:17 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

> As per https://github.com/openjdk/jdk/pull/28002#issuecomment-3789507594 on JDK-8370691 pull request, splitting out portion of PR#28002 into a separate patch.
> 
> In preparation of Float16 vector API support, changes to nomenclature of concrete vector class by suffixing the vector type with vector bit sizes e.g. FloatVector256 so that we don't end up generating names like Float16256Vector.
> 
> All existing Vector API jtreg test are passing with the patch.
> 
> Kindly review and share your feedback.
> 
> Best Regards,
> Jatin

test/jdk/jdk/incubator/vector/gen-tests.sh line 61:

> 59:   args="-K$type -Dtype=$type -DType=$Type -DTYPE=$TYPE"
> 60: 
> 61:   VectorPrefix=$Type

This seems unnecessary - why do we need it?

test/jdk/jdk/incubator/vector/gen-tests.sh line 127:

> 125:   abstractvectortype=${typeprefix}${VectorPrefix}Vector
> 126:   abstractvectorteststype=${typeprefix}${VectorPrefix}VectorTests
> 127:   abstractbitsvectortype=${typeprefix}${VectorPrefixe}Vector

Suggestion:

  abstractbitsvectortype=${typeprefix}${VectorPrefix}Vector

But, why is it changed from ${Bitstype}?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29614#discussion_r2789562537
PR Review Comment: https://git.openjdk.org/jdk/pull/29614#discussion_r2789551075


More information about the core-libs-dev mailing list