On Thu, 4 Mar 2021 18:45:27 GMT, Sandhya Viswanathan <sviswanat...@openjdk.org> 
wrote:

>> Looks good, a nice incremental improvement.
>> 
>> I suppose `checkIndexes` and `wrapIndexes` could call `laneIsValid`, and 
>> then call `anyFalse` on the resulting mask. Dunno if that would affect the 
>> generated code.
>
> Calling laneIsValid from checkIndexes and wrapIndexes would look like as 
> below:
>         VectorMask<E> vecmask = this.laneIsValid();
>         if (!vecmask.allTrue()) {
> I observe a small overhead (~2%) due to couple of extra instructions 
> generated for allTrue.

Ok, I mistakenly thought there was a method `Vector.anyFalse`!

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

PR: https://git.openjdk.java.net/jdk/pull/2819

Reply via email to