On Mon, 8 Dec 2025 10:35:23 GMT, Bhavana Kilambi <[email protected]> wrote:

>> Jatin Bhateja has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Cleanups
>
> test/hotspot/jtreg/compiler/vectorapi/TestFloat16VectorOperations.java line 
> 134:
> 
>> 132:                            .intoArray(output, i);
>> 133:         }
>> 134:         for (; i < LEN; i++) {
> 
> Will this not result in autovectorization instead and also overwrite the 
> `output` array results from vectorapi which were previously computed?

Yes: there could be auto-vectorization.
No: `i` is not reset, it keeps counting from where `i < SPECIES.loopBound(LEN)` 
fails, and handles the tail, right?

It could be good to run this test once with and once without auto 
vectorization, just to make sure the vectors you see are from the Vector API, 
and not auto vectorization.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28002#discussion_r2598054314

Reply via email to