[PATCH] D110177: [AArch64][SVE] Remove redundant PTEST following PNEXT/PFIRST
Sander de Smalen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 21 09:36:15 PDT 2021
sdesmalen added inline comments.
================
Comment at: llvm/test/CodeGen/AArch64/sve-ptest-removal-pfirst.mir:3
+
+# Test instruction sequences where PTEST is redundant and thus gets removed.
+---
----------------
Do these tests necessarily need to be MIR tests? The reason I ask is because I generally find them more difficult to read and would have thought some LLVM IR intrinsics would have been sufficient to prove an extra `ptest` is not added.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110177/new/
https://reviews.llvm.org/D110177
More information about the llvm-commits
mailing list