[llvm] 6d972ad - ControlHeightReduction: Remove assert check in shouldApply

[Matthias Braun via llvm-commits]

Author: Matthias Braun
Date: 2022-10-17T13:10:13-07:00
New Revision: 6d972ad2d8f1cfe22a61d1c4aed6089f0586b377

URL: https://github.com/llvm/llvm-project/commit/6d972ad2d8f1cfe22a61d1c4aed6089f0586b377
DIFF: https://github.com/llvm/llvm-project/commit/6d972ad2d8f1cfe22a61d1c4aed6089f0586b377.diff

LOG: ControlHeightReduction: Remove assert check in shouldApply

Remove assertion checking for non-empty `ProfileSummaryInfo`.

Differential Revision: https://reviews.llvm.org/D133706

Added: 
    

Modified: 
    llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp b/llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp
index f20c1684bed3a..8ecf2cfdcbe13 100644
--- a/llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp
+++ b/llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp
@@ -406,7 +406,6 @@ static bool shouldApply(Function &F, ProfileSummaryInfo& PSI) {
     return CHRFunctions.count(F.getName());
   }
 
-  assert(PSI.hasProfileSummary() && "Empty PSI?");
   return PSI.isFunctionEntryHot(&F);
 }