[PATCH] D22373: [GlobalISel] Introduce a simple instruction selector.

[Tom Stellard via llvm-commits]

tstellarAMD added inline comments.

================
Comment at: lib/CodeGen/GlobalISel/InstructionSelect.cpp:61
@@ +60,3 @@
+         MII != End;) {
+      MachineInstr &MI = *MII++;
+      if (!ISel->select(MI))
----------------
Won't this will fail if the target decides to delete the instruction?  I think targets should be allowed to delete the instruction, because updating MachineIntr in place is very tedious especially if you need to re-order the operands.


https://reviews.llvm.org/D22373