[PATCH] D129629: [llvm-reduce] Fix crash when reducing integer vectors to 1

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/test/tools/llvm-reduce/remove-operands-int.ll:1
+; Test that llvm-reduce can reduce floating point operands
+;
----------------
frasercrmck wrote:
> note: the delta seems to be called "reduce operands" but the floating-point test was called "remove" so I followed suit. Maybe they should both be renamed `reduce-operands-*.ll`?
Probably should be reduce, the operands are replaced, not removed


================
Comment at: llvm/test/tools/llvm-reduce/remove-operands-int.ll:61
+  %add8 = add <2 x i32> %arg2, <i32 1, i32 1>
+  %add9 = add <2 x i32> %arg2, undef
+  ret void
----------------
Nonsplat constant is another case worth testing


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129629