[PATCH] D95655: [AArch64] Adding Neon Sm3 & Sm4 Intrinsics

Tim Northover via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 9 12:07:38 PST 2021


t.p.northover added inline comments.


================
Comment at: clang/test/CodeGen/aarch64-neon-sm4-sm3.c:5
+
+// RUN: not %clang_cc1 -triple aarch64-linux-gnu -target-feature +neon \
+// RUN: -S -emit-llvm -o - %s 2>&1 | FileCheck --check-prefix=CHECK-NO-CRYPTO %s
----------------
apazos wrote:
> The test is written for aarch64 triple, in addition @t.p.northover might want to add arm64 triple checks. Tim what do you think?
Thanks for thinking of us, but generally I don't think it's worth duplicating tests along those lines, it's really difficult to write something that cares about the difference between `arm64` and `aarch64`. 

`linux-gnu` vs `apple-ios` (or others, these days) is more likely to be an issue, but even there I tend to think it's fine to just check one unless there's a specific reason to think it'll affect the targets differently.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95655/new/

https://reviews.llvm.org/D95655



More information about the cfe-commits mailing list