[PATCH] D118259: [AArch64] Adjust aarch64-neon-intrinsics-constrained test and un-XFAIL

John Brawn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 24 10:01:22 PST 2022


john.brawn added a comment.

In D118259#3275297 <https://reviews.llvm.org/D118259#3275297>, @fhahn wrote:

> Does this clang test actually need to check the generated assembly? Shouldn't it be enough to check that the correct intrinsics are generated?

I could remove the CHECK-ASM checks, but the other xyz-constrained.c tests (including non-aarch64 ones) all do this and it feels a bit weird to just do that to this test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118259



More information about the cfe-commits mailing list