I'm worried about the code size with this approach, especially if you have a
nested switch statement.


http://codereview.chromium.org/8499010/diff/5003/src/hydrogen.cc
File src/hydrogen.cc (right):

http://codereview.chromium.org/8499010/diff/5003/src/hydrogen.cc#newcode2887
src/hydrogen.cc:2887:
CHECK_BAILOUT(VisitStatements(clause->statements()));
I think duplicating all clause bodies increases code size too much.
There should be only one body per clause.

http://codereview.chromium.org/8499010/

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

Reply via email to