Revision: 3939
Author: [email protected]
Date: Wed Feb 24 06:02:20 2010
Log: Don't treat this as a trivial expression.

This caused some regressions in Chromium.

BUG=http://crbug.com/36604
Review URL: http://codereview.chromium.org/652228
http://code.google.com/p/v8/source/detail?r=3939

Modified:
 /branches/bleeding_edge/src/ast.h

=======================================
--- /branches/bleeding_edge/src/ast.h   Fri Feb 19 01:01:31 2010
+++ /branches/bleeding_edge/src/ast.h   Wed Feb 24 06:02:20 2010
@@ -930,10 +930,6 @@
     ASSERT(var_ != NULL);  // Variable must be resolved.
     return var()->is_global() || var()->rewrite()->IsLeaf();
   }
-
-  // Reading from a mutable variable is a side effect, but 'this' is
-  // immutable.
-  virtual bool IsTrivial() { return is_this(); }

   bool IsVariable(Handle<String> n) {
     return !is_this() && name().is_identical_to(n);

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

Reply via email to