[llvm] [NFC][DebugInfo] Switch more call-sites to using iterator-insertion (PR #124283)

Jeremy Morse via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 07:27:41 PST 2025


================
@@ -5449,7 +5448,7 @@ static bool removeEmptyCleanup(CleanupReturnInst *RI, DomTreeUpdater *DTU) {
     }
 
     // Sink any remaining PHI nodes directly into UnwindDest.
-    Instruction *InsertPt = DestEHPad;
+    BasicBlock::iterator InsertPt = UnwindDest->getFirstNonPHIIt(); // XXX unsafe faff
----------------
jmorse wrote:

Left over indeed, removing -- it was uh, "unsafe" because I thought some of these call-sites wouldn't recompile cleanly without touching headers, and I was on a laptop which didn't have enough energy left to totally recompile LLVM X_X.

https://github.com/llvm/llvm-project/pull/124283


More information about the llvm-commits mailing list