From 412b3932c6612f4a60bfa8e0811cf6ba1bd3e0e8 Mon Sep 17 00:00:00 2001 From: Evgeniy Brevnov Date: Tue, 28 Jul 2020 19:50:40 +0700 Subject: [PATCH] [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 --- llvm/lib/Analysis/BranchProbabilityInfo.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/Analysis/BranchProbabilityInfo.cpp b/llvm/lib/Analysis/BranchProbabilityInfo.cpp index 195fc69..7e34f3b 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() || -- 2.7.4