This must fail a bunch of tests. The frame state (at least in debug builds) is required to be spilled before calling Load.
LGTM after fixing the spill-state issue and if it passes all the tests. http://codereview.chromium.org/17610/diff/1/2 File src/codegen-ia32.cc (right): http://codereview.chromium.org/17610/diff/1/2#newcode4139 Line 4139: VirtualFrame::SpilledScope spilled_scope(this); This spill has to be eliminated. http://codereview.chromium.org/17610 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
