Reviewers: Sven Panne,

Message:
Hi Sven, a small cleanup since "deferred slot processing" was removed from type
vectors. PTAL, thx,
--Michael

Description:
Small cleanup: AstConstructionVisitor no longer needs a Zone pointer.

R=svenpa...@chromium.org

Please review this at https://codereview.chromium.org/265623002/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+2, -9 lines):
  M src/ast.h


Index: src/ast.h
diff --git a/src/ast.h b/src/ast.h
index d6af25e37dfcee64dbe163a114caa27fc92947a8..0115d988272c44e64f326d6f7377619824f2ea41 100644
--- a/src/ast.h
+++ b/src/ast.h
@@ -2885,9 +2885,7 @@ private: \

 class AstConstructionVisitor BASE_EMBEDDED {
  public:
-  explicit AstConstructionVisitor(Zone* zone)
-    : dont_optimize_reason_(kNoReason),
-      zone_(zone) { }
+  AstConstructionVisitor() : dont_optimize_reason_(kNoReason) { }

   AstProperties* ast_properties() { return &properties_; }
   BailoutReason dont_optimize_reason() { return dont_optimize_reason_; }
@@ -2917,14 +2915,11 @@ class AstConstructionVisitor BASE_EMBEDDED {

   AstProperties properties_;
   BailoutReason dont_optimize_reason_;
-  Zone* zone_;
 };


 class AstNullVisitor BASE_EMBEDDED {
  public:
-  explicit AstNullVisitor(Zone* zone) {}
-
   // Node visitors.
 #define DEF_VISIT(type) \
   void Visit##type(type* node) {}
@@ -2940,9 +2935,7 @@ class AstNullVisitor BASE_EMBEDDED {
 template<class Visitor>
 class AstNodeFactory V8_FINAL BASE_EMBEDDED {
  public:
-  explicit AstNodeFactory(Zone* zone)
-    : zone_(zone),
-      visitor_(zone) { }
+  explicit AstNodeFactory(Zone* zone) : zone_(zone) { }

   Visitor* visitor() { return &visitor_; }



--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to