Revision: 8499
Author: [email protected]
Date: Thu Jun 30 08:49:14 2011
Log: Remove bogus assertion.
[email protected]
Review URL: http://codereview.chromium.org/7291001
http://code.google.com/p/v8/source/detail?r=8499
Modified:
/branches/bleeding_edge/src/ia32/macro-assembler-ia32.cc
/branches/bleeding_edge/src/x64/macro-assembler-x64.cc
=======================================
--- /branches/bleeding_edge/src/ia32/macro-assembler-ia32.cc Tue Jun 28
08:22:08 2011
+++ /branches/bleeding_edge/src/ia32/macro-assembler-ia32.cc Thu Jun 30
08:49:14 2011
@@ -1766,17 +1766,14 @@
mov(dst, esi);
}
- // We should not have found a with or catch context by walking the
context
- // chain (i.e., the static scope chain and runtime context chain do not
- // agree). A variable occurring in such a scope should have slot type
- // LOOKUP and not CONTEXT.
+ // We should not have found a with context by walking the context chain
+ // (i.e., the static scope chain and runtime context chain do not agree).
+ // A variable occurring in such a scope should have slot type LOOKUP and
+ // not CONTEXT.
if (emit_debug_code()) {
cmp(FieldOperand(dst, HeapObject::kMapOffset),
isolate()->factory()->with_context_map());
Check(not_equal, "Variable resolved to with context.");
- cmp(FieldOperand(dst, HeapObject::kMapOffset),
- isolate()->factory()->with_context_map());
- Check(not_equal, "Variable resolved to catch context.");
}
}
=======================================
--- /branches/bleeding_edge/src/x64/macro-assembler-x64.cc Tue Jun 28
08:22:08 2011
+++ /branches/bleeding_edge/src/x64/macro-assembler-x64.cc Thu Jun 30
08:49:14 2011
@@ -3628,17 +3628,14 @@
movq(dst, rsi);
}
- // We should not have found a with or catch context by walking the
context
- // chain (i.e., the static scope chain and runtime context chain do not
- // agree). A variable occurring in such a scope should have slot type
- // LOOKUP and not CONTEXT.
+ // We should not have found a with context by walking the context
+ // chain (i.e., the static scope chain and runtime context chain do
+ // not agree). A variable occurring in such a scope should have
+ // slot type LOOKUP and not CONTEXT.
if (emit_debug_code()) {
CompareRoot(FieldOperand(dst, HeapObject::kMapOffset),
Heap::kWithContextMapRootIndex);
Check(not_equal, "Variable resolved to with context.");
- CompareRoot(FieldOperand(dst, HeapObject::kMapOffset),
- Heap::kCatchContextMapRootIndex);
- Check(not_equal, "Variable resolved to catch context.");
}
}
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
