Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(184)

Unified Diff: src/parsing/parser.cc

Issue 2650703003: [parser]: Skipping inner funcs / initial implemetation of storing scope analysis data from preparse… (Closed)
Patch Set: rebased - argh, forced to format build.gn Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/parsing/parser.h ('k') | src/parsing/preparsed-scope-data.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/parsing/parser.cc
diff --git a/src/parsing/parser.cc b/src/parsing/parser.cc
index 901e49d0b0237dd5594a58a9eaeb49ae4d1f7fb2..96fedf5374ad517112ef8e9dfd7e56afe83bbeb0 100644
--- a/src/parsing/parser.cc
+++ b/src/parsing/parser.cc
@@ -519,7 +519,8 @@ Parser::Parser(ParseInfo* info)
cached_parse_data_(nullptr),
total_preparse_skipped_(0),
temp_zoned_(false),
- log_(nullptr) {
+ log_(nullptr),
+ preparsed_scope_data_(info->preparsed_scope_data()) {
// Even though we were passed ParseInfo, we should not store it in
// Parser - this makes sure that Isolate is not accidentally accessed via
// ParseInfo during background parsing.
@@ -2685,7 +2686,8 @@ FunctionLiteral* Parser::ParseFunctionLiteral(
// If the preconditions are correct the function body should never be
// accessed, but do this anyway for better behaviour if they're wrong.
body = nullptr;
- scope->AnalyzePartially(&previous_zone_ast_node_factory);
+ scope->AnalyzePartially(&previous_zone_ast_node_factory,
+ preparsed_scope_data_);
}
DCHECK_IMPLIES(use_temp_zone, temp_zoned_);
« no previous file with comments | « src/parsing/parser.h ('k') | src/parsing/preparsed-scope-data.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698