[llvm] 241a05a - Revert "[Inliner] Fix bugs for partial inlining with vector"

[via llvm-commits]

Author: joshua-arch1
Date: 2024-08-01T20:16:36+08:00
New Revision: 241a05af0c2324eb073747d42466b9dde229655f

URL: https://github.com/llvm/llvm-project/commit/241a05af0c2324eb073747d42466b9dde229655f
DIFF: https://github.com/llvm/llvm-project/commit/241a05af0c2324eb073747d42466b9dde229655f.diff

LOG: Revert "[Inliner] Fix bugs for partial inlining with vector"

This reverts commit https://github.com/llvm/llvm-project/commit/0a5e5728fbb61d7c775aabc8048b7b629e9ca2d2,
since I forgot to start a pull request.

Added: 
    

Modified: 
    llvm/lib/Transforms/IPO/PartialInlining.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/IPO/PartialInlining.cpp b/llvm/lib/Transforms/IPO/PartialInlining.cpp
index effdb17370201..3ca095e1520f3 100644
--- a/llvm/lib/Transforms/IPO/PartialInlining.cpp
+++ b/llvm/lib/Transforms/IPO/PartialInlining.cpp
@@ -1308,8 +1308,8 @@ bool PartialInlinerImpl::tryPartialInline(FunctionCloner &Cloner) {
   InstructionCost SizeCost = std::get<0>(OutliningCosts);
   InstructionCost NonWeightedRcost = std::get<1>(OutliningCosts);
 
-  if (!SizeCost.isValid() || !NonWeightedRcost.isValid())
-    return false;
+  assert(SizeCost.isValid() && NonWeightedRcost.isValid() &&
+         "Expected valid costs");
 
   // Only calculate RelativeToEntryFreq when we are doing single region
   // outlining.