Reviewers: fschneider,

Message:
The original patch is in patch set 1 and 2 and patch set 3 is the changes to
record the position. Patch set 4 is just a rebase.

Description:
Increase coverage of global loads in optimized code

In the cases where a global property cell cannot be used in the optimized code
use standard load ic to get the property instead of bailing out.

This is re-committing r7212 and r7215 which where reverted in r7239 with the
addition of recoring the source position in the hydrogen code for the
LoadGlobalCell instruction. To record that position an optional position field
has been added to the variable proxy AST node.

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

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/arm/full-codegen-arm.cc
  M     src/arm/lithium-arm.h
  M     src/arm/lithium-arm.cc
  M     src/arm/lithium-codegen-arm.cc
  M     src/ast.h
  M     src/ast.cc
  M     src/checks.h
  M     src/hydrogen-instructions.h
  M     src/hydrogen-instructions.cc
  M     src/hydrogen.h
  M     src/hydrogen.cc
  M     src/ia32/full-codegen-ia32.cc
  M     src/ia32/lithium-codegen-ia32.cc
  M     src/ia32/lithium-ia32.h
  M     src/ia32/lithium-ia32.cc
  M     src/parser.cc
  M     src/scopes.h
  M     src/scopes.cc
  M     src/x64/full-codegen-x64.cc
  M     src/x64/lithium-codegen-x64.cc
  M     src/x64/lithium-x64.h
  M     src/x64/lithium-x64.cc
  A     test/mjsunit/compiler/global-accessors.js


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

Reply via email to