[PATCH] D128192: [GlobalISel][DebugInfo] Propagate debug location for localized constants

Vladislav Dzhidzhoev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 8 08:30:54 PDT 2022


dzhidzhoev added inline comments.


================
Comment at: llvm/lib/CodeGen/GlobalISel/Localizer.cpp:182
     MI->removeFromParent();
-    MBB.insert(II, MI);
+    auto NewMI = MBB.insert(II, MI);
     Changed = true;
----------------
arsenm wrote:
> Isn't NewMI the same as MI?
It seems that MI is copied on insert call.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128192



More information about the llvm-commits mailing list