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

Unified Diff: src/parsing/parse-info.h

Issue 2645613008: Revert of [Parse] ParseInfo owns the parsing Zone. (Closed)
Patch Set: 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/debug/debug-scopes.cc ('k') | src/parsing/parse-info.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/parsing/parse-info.h
diff --git a/src/parsing/parse-info.h b/src/parsing/parse-info.h
index a5fd1a55d333076f4a70e86947bffcce9aea00da..87052a5a7e1d053341658a12cb63f458d1e014c7 100644
--- a/src/parsing/parse-info.h
+++ b/src/parsing/parse-info.h
@@ -4,8 +4,6 @@
#ifndef V8_PARSING_PARSE_INFO_H_
#define V8_PARSING_PARSE_INFO_H_
-
-#include <memory>
#include "include/v8.h"
#include "src/globals.h"
@@ -18,7 +16,6 @@
namespace internal {
-class AccountingAllocator;
class AstRawString;
class AstValueFactory;
class DeclarationScope;
@@ -32,18 +29,13 @@
// A container for the inputs, configuration options, and outputs of parsing.
class V8_EXPORT_PRIVATE ParseInfo {
public:
- explicit ParseInfo(AccountingAllocator* zone_allocator);
- ParseInfo(Handle<Script> script);
- ParseInfo(Handle<SharedFunctionInfo> shared);
-
- // TODO(rmcilroy): Remove once Hydrogen no longer needs this.
- ParseInfo(Handle<SharedFunctionInfo> shared, std::shared_ptr<Zone> zone);
+ explicit ParseInfo(Zone* zone);
+ ParseInfo(Zone* zone, Handle<Script> script);
+ ParseInfo(Zone* zone, Handle<SharedFunctionInfo> shared);
~ParseInfo();
- Zone* zone() const { return zone_.get(); }
-
- std::shared_ptr<Zone> zone_shared() const { return zone_; }
+ Zone* zone() const { return zone_; }
// Convenience accessor methods for flags.
#define FLAG_ACCESSOR(flag, getter, setter) \
@@ -232,7 +224,7 @@
};
//------------- Inputs to parsing and scope analysis -----------------------
- std::shared_ptr<Zone> zone_;
+ Zone* zone_;
unsigned flags_;
ScriptCompiler::ExternalSourceStream* source_stream_;
ScriptCompiler::StreamedSource::Encoding source_stream_encoding_;
« no previous file with comments | « src/debug/debug-scopes.cc ('k') | src/parsing/parse-info.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698