[PATCH] D81956: GlobalISel: Fix some artifact combiner worklist inconsistencies

[Matt Arsenault via Phabricator via llvm-commits]

arsenm marked an inline comment as done.
arsenm added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:755
+          for (const MachineOperand &Def : MI.defs())
+            UpdatedDefs.push_back(Def.getReg());
           break;
----------------
aemerson wrote:
> A G_MERGE should never have multiple defs?
This is handling G_UNMERGE_VALUES, not merge


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

https://reviews.llvm.org/D81956