[PATCH] D125212: [GlobalISel] Implement the HasNoUse builtin predicate

[Abinav Puthan Purayil via Phabricator via llvm-commits]

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


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/InstructionSelectorImpl.h:390
+      const MachineInstr *MI = State.MIs[InsnID];
+      assert(MI != nullptr && "Used insn before defined");
+      assert(MI->getNumDefs() > 0 && "No defs");
----------------
foad wrote:
> Don't need `== nullptr`.
Don't we need to make sure that State.MIs[InsnID] is not a nullptr here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125212