[turbofan] escape analysis supports arguments object and rest elements

src/compiler/escape-analysis-reducer.h

Index: src/compiler/escape-analysis-reducer.h
diff --git a/src/compiler/escape-analysis-reducer.h b/src/compiler/escape-analysis-reducer.h
index ff54d731b7d8d59fe180e075123d8405b69628c7..4373fa4c66a8cb15eee558310fe87ae2a76418e4 100644
--- a/src/compiler/escape-analysis-reducer.h
+++ b/src/compiler/escape-analysis-reducer.h
@@ -26,6 +26,8 @@ class V8_EXPORT_PRIVATE EscapeAnalysisReducer final
 
   Reduction Reduce(Node* node) final;
 
+  void Finalize() override;
+
   // Verifies that all virtual allocation nodes have been dealt with. Run it
   // after this reducer has been applied. Has no effect in release mode.
   void VerifyReplacement() const;
@@ -58,6 +60,7 @@ class V8_EXPORT_PRIVATE EscapeAnalysisReducer final
   // and nodes that do not need a visit from ReduceDeoptState etc.
   BitVector fully_reduced_;
   bool exists_virtual_allocate_;
+  std::set<Node*> arguments_elements_;

[Tobias Tebbi, 2017/02/15 14:16:05]

This is a small set, at most 2 elements, and only constructed once per
compilation. Is this OK, or does it have to be zone memory?

   bool compilation_failed_ = false;
 
   DISALLOW_COPY_AND_ASSIGN(EscapeAnalysisReducer);