[PATCH] D132116: Precommit tests for D132115
David Sherwood via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 18 02:47:46 PDT 2022
david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.
LGTM!
================
Comment at: llvm/test/CodeGen/AArch64/sve-vscale-combine.ll:93
+
+define i32 @multiple_uses_sub_vscale_i32(i32 %x, i32 %y) nounwind {
+; CHECK-LABEL: multiple_uses_sub_vscale_i32:
----------------
nit: I think this test is redundant as it's really testing the same thing as the i64 test. Perhaps you can remove this one before landing the patch?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132116/new/
https://reviews.llvm.org/D132116
More information about the llvm-commits
mailing list