On Sun, 1 Feb 2026 07:36:35 GMT, Jatin Bhateja <[email protected]> wrote:

>> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/AbstractSpecies.java
>>  line 152:
>> 
>>> 150:     int laneTypeOrdinal() {
>>> 151:         return laneType.ordinal();
>>> 152:     }
>> 
>> Is this needed? Won't all concrete sub types override this?
>
> This interface provides access to lane type constant though species, its used 
> for consistency, please have a look at following line and other places around 
> it.
> https://github.com/jatin-bhateja/jdk/blob/ff73dc3d48a9435c4395556c8325fbce7610cba9/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/DoubleVector.java#L3374

Agreed that this method is required, but i was wondering why `AbstractSpecies` 
need to implement it. Ok, i see now you are copying the same pattern as some 
other methods such as `elementType`, so this is a more general issue we should 
not resolve in this PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29481#discussion_r2751614740

Reply via email to