[PATCH] D57478: GlobalISel: Implement fewerElementsVector for shifts

[Matt Arsenault via Phabricator via llvm-commits]

arsenm marked 2 inline comments as done.
arsenm added inline comments.


================
Comment at: lib/CodeGen/GlobalISel/LegalizerHelper.cpp:1850
+                      LeftoverRegs))
+      llvm_unreachable("should have been able to extractParts");
+
----------------
aemerson wrote:
> We can return 0 here to fall back instead of dying.
Is it OK I fa failed legalization created dead instructions?


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

https://reviews.llvm.org/D57478