Reviewers: Jakob,

Message:
This should be reviewable: all tests pass and also more complex code executes
correctly.

Description:
Array index computation dehoisting.

When an array index (in an array access) is a simple "expression + constant",
just embed the constant in the array access operation so that the full index
expression is (potentially) no longer used and its live range can be much
shorter.
This is effective in conjunction with array bounds check removal (otherwise the
index is anyway used in the check).


Please review this at https://chromiumcodereview.appspot.com/10382055/

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

Affected files:
  M src/arm/lithium-arm.h
  M src/arm/lithium-arm.cc
  M src/arm/lithium-codegen-arm.cc
  M src/flag-definitions.h
  M src/hydrogen-instructions.h
  M src/hydrogen.h
  M src/hydrogen.cc
  M src/ia32/lithium-codegen-ia32.h
  M src/ia32/lithium-codegen-ia32.cc
  M src/ia32/lithium-ia32.h
  M src/ia32/lithium-ia32.cc
  M src/v8-counters.h
  M src/x64/lithium-codegen-x64.h
  M src/x64/lithium-codegen-x64.cc
  M src/x64/lithium-x64.h
  M src/x64/lithium-x64.cc


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

Reply via email to