[PATCH] D132116: Precommit tests for D132115

WangLian via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 18 02:50:16 PDT 2022


Jimerlife added inline comments.


================
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:
----------------
david-arm wrote:
> 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?
OK, I will remove this test. Thank you.


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