Merge pull request #21897 from shajrawi/excl_inline

[PerformanceInliner] Reduce the code size impact of the exclusivity heuristic

Author: Joe Shajrawi

lib/SILOptimizer/Transforms/PerformanceInliner.cpp

@@ -38,6 +38,10 @@ llvm::cl::opt<bool> EnableSILInliningOfGenerics(
   "sil-inline-generics", llvm::cl::init(false),
   llvm::cl::desc("Enable inlining of generics"));
 
+llvm::cl::opt<bool>
+    EnableSILAgressiveInlining("sil-agressive-inline", llvm::cl::init(false),
+                               llvm::cl::desc("Enable agressive inlining"));
+
 //===----------------------------------------------------------------------===//
 //                           Performance Inliner
 //===----------------------------------------------------------------------===//
@@ -105,7 +109,7 @@ class SILPerformanceInliner {
     /// The benefit of inlining an exclusivity-containing callee.
     /// The exclusivity needs to be: dynamic,
     /// has no nested conflict and addresses known storage
-    ExclusivityBenefit = RemovedCallBenefit + 125,
+    ExclusivityBenefit = RemovedCallBenefit + 10,
 
     /// The benefit of inlining class methods with -Osize.
     /// We only inline very small class methods with -Osize.
@@ -321,6 +325,10 @@ bool SILPerformanceInliner::isProfitableToInline(
   // the exclusivity heuristic or not. We can *only* do that
   // if AllAccessesBeneficialToInline is true
   int ExclusivityBenefitWeight = 0;
+  int ExclusivityBenefitBase = ExclusivityBenefit;
+  if (EnableSILAgressiveInlining) {
+    ExclusivityBenefitBase += 500;
+  }
 
   SubstitutionMap CalleeSubstMap = AI.getSubstitutionMap();
 
@@ -422,7 +430,8 @@ bool SILPerformanceInliner::isProfitableToInline(
           if (BAI->hasNoNestedConflict() &&
               (storage.isUniquelyIdentified() ||
                storage.getKind() == AccessedStorage::Class)) {
-            BlockW.updateBenefit(ExclusivityBenefitWeight, ExclusivityBenefit);
+            BlockW.updateBenefit(ExclusivityBenefitWeight,
+                                 ExclusivityBenefitBase);
           } else {
             AllAccessesBeneficialToInline = false;
           }