Relax inlining limits for simple leaf functions.

src/hydrogen.h

Index: src/hydrogen.h
diff --git a/src/hydrogen.h b/src/hydrogen.h
index 948ea342bce660e12c8e14067b4b305188317560..e81f338456864e287acd7a28dc2ebe13128725e2 100644
--- a/src/hydrogen.h
+++ b/src/hydrogen.h
@@ -752,8 +752,12 @@ class HGraphBuilder: public AstVisitor {
   static const int kMaxLoadPolymorphism = 4;
   static const int kMaxStorePolymorphism = 4;
 
-  static const int kMaxInlinedNodes = 196;
+
+  // Make sure that kMaxInlinedSize < SharedFunctionInfo::kAstNodeCountMask.

[Kevin Millikin (Chromium), 2011/11/28 12:11:59]

How about:

STATIC_ASSERT(kMaxInlinedSize < SharedFunctionInfo::kAstNodeCountMask);

[ulan, 2011/11/29 14:06:52]

Done.

+  static const int kMaxInlinedNodesSoft = 196;
+  static const int kMaxInlinedNodesHard = 270;

[Kevin Millikin (Chromium), 2011/11/28 12:11:59]

I still don't think these constants are documented very well, here or in the
source.  I suggest something like:

// Hard limits.
static const int kMaxSourceSize = 600;  // Per site.
static const int kMaxInlinedSize = 196;  // Per site.
static const int kMaxInlinedNodesHard = 270;  // Cumulative.

// Soft limits.  After the soft limit is reached we will only inline small
primitives.
static const int kMaxInlinedNodesSoft = 196;  // Cumulative
static const int kMaxInlinedPrimitiveSize = 40;  // Per site.

[ulan, 2011/11/29 14:06:52]

Done.

   static const int kMaxInlinedSize = 196;
+  static const int kMaxInlinedPrimitiveSize = 40;
   static const int kMaxSourceSize = 600;
 
   // Simple accessors.