Do not change environment between simulate and scope with no observable side-effects in HandlePrope…

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

Committed: https://code.google.com/p/v8/source/detail?r=14584

[ulan, 2013-05-07 16:02:32]

PTAL.

I will add verification of environment in a separate CL.

[danno, 2013-05-07 16:53:33]

lgtm

[ulan, 2013-05-08 07:40:35]

Committed patchset #1 manually as r14584 (presubmit successful).