Chromium Code Reviews

Unified Diff: src/prettyprinter.cc

Issue 1306993003: Call JS functions via native context instead of js builtins object. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: separated context slot lookup Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: src/prettyprinter.cc
diff --git a/src/prettyprinter.cc b/src/prettyprinter.cc
index e71489fa7f2c401111468f7de94d55bf24075a4a..939f5745a76f4b9ea2f6b7bacf538d912b80c63d 100644
--- a/src/prettyprinter.cc
+++ b/src/prettyprinter.cc
@@ -795,8 +795,7 @@ void PrettyPrinter::VisitCallNew(CallNew* node) {
void PrettyPrinter::VisitCallRuntime(CallRuntime* node) {
- Print("%%");
- PrintLiteral(node->name(), false);
+ Print("%%%s\n", node->function()->name);
Michael Starzinger 2015/08/26 09:43:22 Is node guaranteed to have a runtime function or i
Yang 2015/08/26 10:27:30 Done.
PrintArguments(node->arguments());
}
@@ -1523,9 +1522,8 @@ void AstPrinter::VisitCallNew(CallNew* node) {
void AstPrinter::VisitCallRuntime(CallRuntime* node) {
EmbeddedVector<char, 128> buf;
- FormatICSlotNode(&buf, node, "CALL RUNTIME", node->CallRuntimeFeedbackSlot());
IndentedScope indent(this, buf.start());
- PrintLiteralIndented("NAME", node->name(), false);
+ Print("NAME %s\n", node->function()->name);
Michael Starzinger 2015/08/26 09:43:22 Likewise.
Yang 2015/08/26 10:27:30 Done.
PrintArguments(node->arguments());
}

Powered by Google App Engine