Reviewers: Mads Ager,

http://codereview.chromium.org/6685119/diff/1/src/ia32/full-codegen-ia32.cc
File src/ia32/full-codegen-ia32.cc (right):

http://codereview.chromium.org/6685119/diff/1/src/ia32/full-codegen-ia32.cc#newcode1138
src/ia32/full-codegen-ia32.cc:1138: ASSERT((var->is_global() &&
!var->is_this()) == (slot == NULL));
This was a silly typo, only present on the ia32 platform.

Description:
[Arguments] Remove the arguments shadow symbol.

The only essential use remaining was to detect an allocated arguments object
to avoid OSR.  The shared function info now has a flag indicating that there
might be an allocated arguments object so we can avoid OSR for those
functions.

[email protected]


Please review this at http://codereview.chromium.org/6685119/

SVN Base: https://v8.googlecode.com/svn/branches/experimental/arguments

Affected files:
  M src/compiler.cc
  M src/contexts.cc
  M src/heap-profiler.h
  M src/heap.h
  M src/ia32/full-codegen-ia32.cc
  M src/objects-inl.h
  M src/objects.h
  M src/runtime-profiler.cc
  M src/runtime.cc
  M src/scopeinfo.h


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

Reply via email to