[aarch64-port-dev ] RFR(S): 8214922: Add vectorization support for fmin/fmax

[Andrew Haley]

Hi,

On 12/21/18 12:00 PM, Roland Westrelin wrote:
> 
>> Webrev: http://cr.openjdk.java.net/~pli/rfr/8214922/webrev.00/
> 
>  600     case Op_MinF:
>  601       assert(bt == T_FLOAT, "must be");
>  602       if (Matcher::max_vector_size(T_FLOAT) > 2) {
>  603         vopc = Op_MinReductionV;
>  604       }
>  605       break;
> 
> Why is the Matcher::max_vector_size(T_FLOAT) > 2 check needed?

Code quality is worse, apparently.

-- 
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671