RFR: 8328181: C2: assert(MaxVectorSize >= 32) failed: vector length should be >= 32 [v2]

[Jatin Bhateja]

On Tue, 26 Mar 2024 16:40:31 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Cleanup predicates.
>
> src/hotspot/cpu/x86/x86.ad line 1755:
> 
>> 1753:     case Op_ClearArray:
>> 1754:       if ((size_in_bits != 512) && !VM_Version::supports_avx512vl()) {
>> 1755:         return false;
> 
> Please add comment to clarify condition. I am reading it as ClearArray will not be supported for NOT avx512 because we can have vector length 512 bits for not avx512.

This is only pertinent to known sized clear arrays which are optimized for AVX-512 targets, we already have such a check as part of matcher predicate, so removing it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18464#discussion_r1555163994