[vectorIntrinsics] RFR: 8263149: Adding some algorithms optimized by Vector API into JMH benc… [v2]

[Wang Zhuo]

On Tue, 9 Mar 2021 03:33:17 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> Wang Zhuo has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>> 
>>   8263149: Adding some algorithms optimized by Vector API into JMH benchmarks
>>   
>>   Co-authored-by: Joshua Zhu <jzhu at openjdk.org>
>
> test/jdk/jdk/incubator/vector/benchmark/src/main/java/benchmark/bigdata/VectorDistance.java line 128:
> 
>> 126:         for (i = 0; i + (SPECIES.length()) <= queryVectorFloat.length; i += SPECIES.length()) {
>> 127:             vecX = FloatVector.fromArray(SPECIES, queryVectorFloat, i);
>> 128:             vecY = FloatVector.fromArray(SPECIES, inputVectorFloat, i);
> 
> Thanks for your JMH benchmarks! Could you please use `"SPECIES_FLOAT_128"` inside the API instead of defining a new variable here? I'm wondering whether the API can be vectorized by using `"SPECIES"` here, since the API needs the SPECIES argument to be a constant. I'm not sure about this case, but I really met the same issue before when I ran the jtreg tests. To make sure everything works as expect,  I'd like using the `static final field "SPECIES_FLOAT_128"` all through the API usage.

done

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

PR: https://git.openjdk.java.net/panama-vector/pull/45