Having read through a few times, I have some high-level questions:

- You mentioned on IRC that the keeping-track-of-assigments-to-rewrite thing
didn't interact well with something else. Was that resolved? Overall, do you
know of any cases where this doesn't work correctly?

- It seems a bit strange that the AssignmentPattern AST node subsumes the entire Assignment. This makes the code in full-codegen look odd, since only the target
is visited and not the value. Was there a reason you didn't put more of this
logic in Assignment?

https://codereview.chromium.org/1309813007/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to