On Wed, 20 Aug 2025 06:49:41 GMT, Emanuel Peter <[email protected]> wrote:
>> Galder Zamarreño has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Check at the very least that auto vectorization is supported
>
> test/micro/org/openjdk/bench/vm/compiler/VectorBitConversion.java line 90:
>
>> 88:
>> 89: @Benchmark
>> 90: public long[] doubleToLongBits() {
>
> I wonder if we should not just extend this benchmark, that has `convertI2F`
> etc:
> `test/micro/org/openjdk/bench/vm/compiler/TypeVectorOperations.java`
>
> Just a suggestion, we can also keep them separately. Maybe one day we should
> clean up the benchmarks, and put them all in some `autovectorization`
> subdirectory, and organize the files and benchmarks a little better.
I'll look into `TypeVectorOperations` and see what can be done there. I had
missed it when I wrote the benchmark.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26457#discussion_r2288560996
