RFR: 8373026: C2 SuperWord and Vector API: vector algorithms test and benchmark [v11]
Xiaohong Gong
xgong at openjdk.org
Thu Jan 22 02:11:32 UTC 2026
On Wed, 21 Jan 2026 09:47:22 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> test/hotspot/jtreg/compiler/vectorization/TestVectorAlgorithms.java line 429:
>>
>>> 427: IRNode.ADD_VI, "> 0",
>>> 428: IRNode.STORE_VECTOR, "> 0"},
>>> 429: applyIfCPUFeatureOr = {"sse4.1", "true", "asimd", "true"},
>>
>> Is this better?
>>
>> Suggestion:
>>
>> applyIfCPUFeatureOr = {"avx512", "true", "sve", "true"},
>
> Are you getting failures with `sse4.1` or `asimd`? Or what is the reason for weakening the conditions here?
There is no failures with these two features because we have checks of `MaxVectorSize` as well. I was thinking that only `avx512` and `sve` CPUs would match the condition `MaxVectorSize >= 64`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28639#discussion_r2715040988
More information about the hotspot-compiler-dev
mailing list