Revision: 9465
Author:   [email protected]
Date:     Wed Sep 28 04:58:40 2011
Log:      Remove HEnvironment::CheckDepth.

[email protected]
BUG=v8:1727

Review URL: http://codereview.chromium.org/8066005
http://code.google.com/p/v8/source/detail?r=9465

Modified:
 /branches/bleeding_edge/src/hydrogen.cc
 /branches/bleeding_edge/src/hydrogen.h

=======================================
--- /branches/bleeding_edge/src/hydrogen.cc     Wed Sep 28 04:34:54 2011
+++ /branches/bleeding_edge/src/hydrogen.cc     Wed Sep 28 04:58:40 2011
@@ -6474,7 +6474,6 @@
   pop_count_ = other->pop_count_;
   push_count_ = other->push_count_;
   ast_id_ = other->ast_id_;
-  CheckDepth();
 }


=======================================
--- /branches/bleeding_edge/src/hydrogen.h      Tue Sep 27 05:55:43 2011
+++ /branches/bleeding_edge/src/hydrogen.h      Wed Sep 28 04:58:40 2011
@@ -448,23 +448,6 @@
  private:
   explicit HEnvironment(const HEnvironment* other);

-  void CheckDepth() {
-    // Verify that we are not trying to create an
-    // impossibly deeply nested environment.
-    if (!FLAG_limit_inlining) return;
-
-    static const int kMaxDepth = 4;
-
-    int cnt = 0;
-    for (HEnvironment* env = this;
- env != NULL && cnt <= kMaxDepth; // Check cnt to avoid infinite loop.
-         env = env->outer()) {
-      cnt++;
-    }
-
-    CHECK(cnt <= kMaxDepth);
-  }
-
// True if index is included in the expression stack part of the environment.
   bool HasExpressionAt(int index) const;

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to