[PATCH] D106879: GlobalISel: Fix infinite loop in legalization artifact combiner

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 27 12:24:29 PDT 2021


aemerson added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:770
         Register DefReg = MI.getReg(DefIdx);
+        if (MRI.use_empty(DefReg))
+          continue;
----------------
arsenm wrote:
> Probably should be use_nodbg_empty
If the def is dead, we should also set the dead bit in `DeadDefs`.


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

https://reviews.llvm.org/D106879



More information about the llvm-commits mailing list