Avoid multiple failures to inline functions

src/hydrogen.cc

Index: src/hydrogen.cc
diff --git a/src/hydrogen.cc b/src/hydrogen.cc
index 3f9d32de1c5ee3153026bb218fdf48f1e40e5a8a..f34814366f65162a7db57c88c3b1bfd21b0d4234 100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -7850,10 +7850,12 @@ bool HOptimizedGraphBuilder::TryInline(Handle<JSFunction> target,
   // earlier the information might not have been complete due to lazy parsing.
   nodes_added = function->ast_node_count();
   if (nodes_added > Min(FLAG_max_inlined_nodes, kUnlimitedMaxInlinedNodes)) {
+    target_shared->set_ast_node_count(nodes_added);

[Michael Starzinger, 2014/10/27 13:27:48]

This should now be superseded by https://codereview.chromium.org/675493002/,
correct?

[wingo, 2014/10/27 15:06:51]

Yes indeed.

     TraceInline(target, caller, "target AST is too large [late]");
     return false;
   }
   if (function->dont_optimize()) {
+    target_shared->set_dont_optimize_reason(function->dont_optimize_reason());

[Michael Starzinger, 2014/10/27 13:27:48]

This scares me, could we instead make it go through
SharedFunctionInfo::DisableOptimization?

[wingo, 2014/10/27 15:06:51]

So this part of the CL is superseded by
https://codereview.chromium.org/668143003, but I guess the comment applies there
too:

  https://codereview.chromium.org/668143003/diff/60001/src/compiler.cc

I'll add you to Cc there and fix that patch.

     TraceInline(target, caller, "target contains unsupported syntax [late]");
     return false;
   }