[vector api] RFR: Implement Vector API andAll/orAll/xorAll for AArch64 NEON
Andrew Haley
aph at redhat.com
Mon Jan 28 10:13:02 UTC 2019
On 1/28/19 2:44 AM, Yang Zhang (Arm Technology China) wrote:
> I have a patch which implements Vector API andAll/orAll/xorAll for AArch64 NEON. Could you please help to review it?
> http://cr.openjdk.java.net/~yzhang/vectorapi.andall/webrev.00/
OK, but:
You're doing a great deal of repetitive stuff in here. This makes the
patch hard to maintain and review. Instead, I suggest that you have a
look at aarch64_ad.m4 to see how it's possible to generate all of
these patterns from a script: 487 lines of that script generates 1970
lines of patterns.
--
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
More information about the panama-dev
mailing list