Do not change environment between simulate and scope with no observable side-effects...
authorulan@chromium.org <ulan@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Wed, 8 May 2013 07:40:28 +0000 (07:40 +0000)
committerulan@chromium.org <ulan@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Wed, 8 May 2013 07:40:28 +0000 (07:40 +0000)
commite5a29e8ff911581967f878226dd267d9d778161a
treebee71ff5beda89b7c94695051e46a54ee970079d
parent064349bae49856cc40c059728f5dc6465b130fff
Do not change environment between simulate and scope with no observable side-effects in HandlePropertyAssignment.

LChunkBuilder reconstructs the environment by applying simulates. A scope with no observable side-effects has no simulates. If the scope deoptimizes, then LChunkBuilder would miss the changes to the environment between the last simulate and the scope.

R=danno@chromium.org
BUG=v8:2671
TEST=mjsunit/regress/regress-2671.js

Review URL: https://chromiumcodereview.appspot.com/14793009

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14584 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
src/hydrogen.cc
test/mjsunit/regress/regress-2671.js [new file with mode: 0644]