Chromium Code Reviews| Index: src/hydrogen.cc |
| =================================================================== |
| --- src/hydrogen.cc (revision 10510) |
| +++ src/hydrogen.cc (working copy) |
| @@ -4798,7 +4798,8 @@ |
| // Do a quick check on source code length to avoid parsing large |
| // inlining candidates. |
| - if (FLAG_limit_inlining && target->shared()->SourceSize() > kMaxSourceSize) { |
| + if ((FLAG_limit_inlining && target->shared()->SourceSize() > kSmallMaxSourceSize) || |
|
Vyacheslav Egorov (Chromium)
2012/01/26 10:42:57
long line
|
| + target->shared()->SourceSize() > kBigMaxSourceSize) { |
| TraceInline(target, caller, "target text too big"); |
| return false; |
| } |
| @@ -4846,7 +4847,8 @@ |
| } |
| // We don't want to add more than a certain number of nodes from inlining. |
| - if (FLAG_limit_inlining && inlined_count_ > kMaxInlinedNodes) { |
| + if ((FLAG_limit_inlining && inlined_count_ > kSmallMaxInlinedNodes) || |
| + inlined_count_ > kBigMaxInlinedNodes) { |
| TraceInline(target, caller, "cumulative AST node limit reached"); |
| return false; |
| } |
| @@ -4874,7 +4876,8 @@ |
| // Count the number of AST nodes added by inlining this call. |
| int nodes_added = AstNode::Count() - count_before; |
| - if (FLAG_limit_inlining && nodes_added > kMaxInlinedSize) { |
| + if ((FLAG_limit_inlining && nodes_added > kSmallMaxInlinedSize) || |
| + nodes_added > kBigMaxInlinedSize) { |
| TraceInline(target, caller, "target AST is too large"); |
| return false; |
| } |