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

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

Issue 2648383005: Revert "[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 c334dffb6b90253327956f63add3c21cccc11eeb..91308183b08d2746f6f20a20787868fc31e2f206 100644
--- a/src/parsing/parse-info.h
+++ b/src/parsing/parse-info.h
@@ -5,8 +5,6 @@
#ifndef V8_PARSING_PARSE_INFO_H_
#define V8_PARSING_PARSE_INFO_H_
-#include <memory>
-
#include "include/v8.h"
#include "src/globals.h"
#include "src/handles.h"
@@ -17,7 +15,6 @@ class Extension;
namespace internal {
-class AccountingAllocator;
class AstRawString;
class AstValueFactory;
class DeclarationScope;
@@ -31,18 +28,13 @@ class Zone;
// 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) \
@@ -231,7 +223,7 @@ class V8_EXPORT_PRIVATE ParseInfo {
};
//------------- 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