Fix a use after free. RI is freed before the call to getDebugLoc(). To
prevent this, capture the location before RI is freed.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@180824 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Transforms/Utils/InlineFunction.cpp b/lib/Transforms/Utils/InlineFunction.cpp
index 019f40d..dabb67b 100644
--- a/lib/Transforms/Utils/InlineFunction.cpp
+++ b/lib/Transforms/Utils/InlineFunction.cpp
@@ -853,11 +853,12 @@
// Add a branch to the merge points and remove return instructions.
- ReturnInst *RI;
+ DebugLoc Loc;
for (unsigned i = 0, e = Returns.size(); i != e; ++i) {
- RI = Returns[i];
+ ReturnInst *RI = Returns[i];
BranchInst* BI = BranchInst::Create(AfterCallBB, RI);
- BI->setDebugLoc(RI->getDebugLoc());
+ Loc = RI->getDebugLoc();
+ BI->setDebugLoc(Loc);
RI->eraseFromParent();
}
// We need to set the debug location to *somewhere* inside the
@@ -865,7 +866,7 @@
// instruction will at least be associated with the right
// function.
if (CreatedBranchToNormalDest)
- CreatedBranchToNormalDest->setDebugLoc(RI->getDebugLoc());
+ CreatedBranchToNormalDest->setDebugLoc(Loc);
} else if (!Returns.empty()) {
// Otherwise, if there is exactly one return value, just replace anything
// using the return value of the call with the computed value.