[PATCH] D85831: [SVE] Lower fixed length vector integer ISD::SETCC operations.

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 12 05:43:58 PDT 2020


paulwalker-arm added reviewers: david-arm, cameron.mcinally.
paulwalker-arm added a comment.

The floating-point side requires some refactoring so I'll do that separately.



================
Comment at: llvm/test/CodeGen/AArch64/sve-fixed-length-int-compares.ll:446
+
+define void @icmp_ne_v32i8(<32 x i8>* %a, <32 x i8>* %b) #0 {
+; CHECK-LABEL: icmp_ne_v32i8:
----------------
I believe I've covered all the combinations for the EQ tests and there should already be tests for the ISEL patterns required for SETCC_MERGE_ZERO.  For these reasons there doesn't look to be a good reason to repeat the vector combinations for each comparison type, so I'm instead just testing a single but different vector length for the following compare tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85831



More information about the llvm-commits mailing list