Reviewers: Michael Starzinger,
Message:
This looks right to me, or at least I can't convince myself it's wrong.
WDYT?
Description:
Avoid multiple failures to inline functions
[email protected]
BUG=
Please review this at https://codereview.chromium.org/668663003/
Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+2, -0 lines):
M 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);
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());
TraceInline(target, caller, "target contains unsupported syntax
[late]");
return false;
}
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
