[llvm-commits] [llvm] r93448 - /llvm/trunk/lib/Analysis/InlineCost.cpp

[Eric Christopher]

Author: echristo
Date: Thu Jan 14 14:12:34 2010
New Revision: 93448

URL: http://llvm.org/viewvc/llvm-project?rev=93448&view=rev
Log:
Reduce the inlining cost of functions that contain calls to easily,
and frequently optimized functions.

Modified:
    llvm/trunk/lib/Analysis/InlineCost.cpp

Modified: llvm/trunk/lib/Analysis/InlineCost.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/InlineCost.cpp?rev=93448&r1=93447&r2=93448&view=diff

==============================================================================
--- llvm/trunk/lib/Analysis/InlineCost.cpp (original)
+++ llvm/trunk/lib/Analysis/InlineCost.cpp Thu Jan 14 14:12:34 2010
@@ -102,6 +102,32 @@
   return Reduction;
 }
 
+// callIsSmall - If a call will lower to a single selection DAG node, or
+// is otherwise deemed small return true.
+// TODO: Perhaps calls like memcpy, strcpy, etc?
+static bool callIsSmall(const Function *F) {
+  if (F && !F->hasLocalLinkage() && F->hasName()) {
+    StringRef Name = F->getName();
+    
+    // These will all likely lower to a single selection DAG node.
+    if (Name == "copysign" || Name == "copysignf" ||
+        Name == "fabs" || Name == "fabsf" || Name == "fabsl" ||
+        Name == "sin" || Name == "sinf" || Name == "sinl" ||
+        Name == "cos" || Name == "cosf" || Name == "cosl" ||
+        Name == "sqrt" || Name == "sqrtf" || Name == "sqrtl" )
+      return true;
+    
+    // These are all likely to be optimized into something smaller.
+    if (Name == "pow" || Name == "powf" || Name == "powl" ||
+        Name == "exp2" || Name == "exp2l" || Name == "exp2f" ||
+        Name == "floor" || Name == "floorf" || Name == "ceil" ||
+        Name == "round" || Name == "ffs" || Name == "ffsl" ||
+        Name == "abs" || Name == "labs" || Name == "llabs")
+      return true;
+  }
+  return false;
+}
+
 /// analyzeBasicBlock - Fill in the current structure with information gleaned
 /// from the specified block.
 void CodeMetrics::analyzeBasicBlock(const BasicBlock *BB) {
@@ -129,7 +155,7 @@
 
       // Calls often compile into many machine instructions.  Bump up their
       // cost to reflect this.
-      if (!isa<IntrinsicInst>(II))
+      if (!isa<IntrinsicInst>(II) && !callIsSmall(CS.getCalledFunction()))
         NumInsts += InlineConstants::CallPenalty;
     }