[PATCH] D70564: [GlobalISel] LegalizationArtifactCombiner: Fix a bug in tryCombineMerges

[Volkan Keles via Phabricator via llvm-commits]

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


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/LegalizationArtifactCombiner.h:277
+    if (llvm::canReplaceReg(DstReg, SrcReg, MRI))
+      MRI.replaceRegWith(DstReg, SrcReg);
+    else
----------------
aditya_nandakumar wrote:
> We should probably be notifying the observer here...
Right, updated the patch.


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

https://reviews.llvm.org/D70564