[llvm] 412b393 - [BPI] Fix memory leak reported by sanitizer bots
Evgeniy Brevnov via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 28 05:54:00 PDT 2020
Author: Evgeniy Brevnov
Date: 2020-07-28T19:53:46+07:00
New Revision: 412b3932c6612f4a60bfa8e0811cf6ba1bd3e0e8
URL: https://github.com/llvm/llvm-project/commit/412b3932c6612f4a60bfa8e0811cf6ba1bd3e0e8
DIFF: https://github.com/llvm/llvm-project/commit/412b3932c6612f4a60bfa8e0811cf6ba1bd3e0e8.diff
LOG: [BPI] Fix memory leak reported by sanitizer bots
There is a silly mistake where release() is used instead of reset() for free resources of unique pointer.
Reviewed By: ebrevnov
Differential Revision: https://reviews.llvm.org/D84747
Added:
Modified:
llvm/lib/Analysis/BranchProbabilityInfo.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Analysis/BranchProbabilityInfo.cpp b/llvm/lib/Analysis/BranchProbabilityInfo.cpp
index 195fc69d9601..7e34f3b6c869 100644
--- a/llvm/lib/Analysis/BranchProbabilityInfo.cpp
+++ b/llvm/lib/Analysis/BranchProbabilityInfo.cpp
@@ -1178,7 +1178,7 @@ void BranchProbabilityInfo::calculate(const Function &F, const LoopInfo &LI,
PostDominatedByUnreachable.clear();
PostDominatedByColdCall.clear();
- SccI.release();
+ SccI.reset();
if (PrintBranchProb &&
(PrintBranchProbFuncName.empty() ||
More information about the llvm-commits
mailing list