LGTM.

http://codereview.chromium.org/114018/diff/1/4
File src/jump-target.cc (right):

http://codereview.chromium.org/114018/diff/1/4#newcode194
Line 194: // address).  Fill those in first.
The new frame does not "have" memory elements for ..., it is
(inaccurately) initialized with memory elements in those slots, and they
need to be replaced.  The comment confuses me as it is written.

http://codereview.chromium.org/114018

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

Reply via email to