On Thu, 4 Mar 2021 18:45:27 GMT, Sandhya Viswanathan <[email protected]>
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
