[vector api] RFR: Implement vector max and min for AArch64 NEON

[Yang Zhang (Arm Technology China)]

Hi Vladimir

>I assume NEON is unconditionally supported on AArch64.
>No need to adjust Matcher::match_rule_supported[_vector] to make use of those new instructions?

Yes, NEON is mandatory feature on AArch64. Usually we don't need to adjust Matcher::match_rule_supported[_vector].

>Otherwise, looks fine to me (not an expert in AArch64 port).

Thanks for your review.

Regards
Yang


-----Original Message-----
From: Vladimir Ivanov <vladimir.x.ivanov at oracle.com> 
Sent: Thursday, November 8, 2018 3:13 AM
To: Yang Zhang (Arm Technology China) <Yang.Zhang at arm.com>; panama-dev at openjdk.java.net
Cc: nd <nd at arm.com>
Subject: Re: [vector api] RFR: Implement vector max and min for AArch64 NEON



On 06/11/2018 23:15, Yang Zhang (Arm Technology China) wrote:
> Hi,
> 
> I have a patch which implements vector api max() and min() for AArch64 NEON.
> Could you please help to review this?
> 
> Webrev:
> http://cr.openjdk.java.net/~yzhang/vectorapi.max_min/webrev.00/

I assume NEON is unconditionally supported on AArch64.

No need to adjust Matcher::match_rule_supported[_vector] to make use of those new instructions?

Otherwise, looks fine to me (not an expert in AArch64 port).

Best regards,
Vladimir Ivanov