[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