RFR: 8261542: X86 slice and unslice intrinsics for 256-bit byte/short vectors [v4]

Vladimir Kozlov kvn at openjdk.java.net
Fri Feb 19 05:52:42 UTC 2021


On Fri, 19 Feb 2021 03:17:56 GMT, Sandhya Viswanathan <sviswanathan at openjdk.org> wrote:

>> My bad - I missed that size is in bytes in assert. The assert is correct, as you said.
>> And `} else {` part works for AVX1 because of match_rule_supported_vector() bailout 256-bit case.
>> May be add assert(UseAVX > 1 || vlen_in_bytes <= 16, ).
>> 
>> I only have one question left - about check >= 256 in  match_rule_supported_vector()
>
> Added the following assert on else path:
> +      assert(UseAVX > 1 || vlen_in_bytes <= 16, "required");

Good.

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

PR: https://git.openjdk.java.net/jdk/pull/2520


More information about the hotspot-dev mailing list