[llvm] [DebugInfo][RemoveDIs] Have getInsertionPtAfterDef return an iterator (PR #73149)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 30 01:50:04 PST 2023


================
@@ -4130,7 +4130,7 @@ static bool canFreelyInvert(InstCombiner &IC, Value *Op,
 static Value *freelyInvert(InstCombinerImpl &IC, Value *Op,
                            Instruction *IgnoredUser) {
   auto *I = cast<Instruction>(Op);
-  IC.Builder.SetInsertPoint(&*I->getInsertionPointAfterDef());
+  IC.Builder.SetInsertPoint(&**I->getInsertionPointAfterDef());
----------------
OCHyams wrote:

The leading `&*` are not needed (were not needed before your patch)? And potentially drops haedBit info, if I'm reading this right.

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


More information about the llvm-commits mailing list