[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