LGTM
http://codereview.chromium.org/6839015/diff/1/src/hydrogen.cc File src/hydrogen.cc (right): http://codereview.chromium.org/6839015/diff/1/src/hydrogen.cc#newcode2277 src/hydrogen.cc:2277: if (scope->function() != NULL) return Bailout("named function expression"); I didn't know this actually works, returning the result of a void function call. Is this something done elsewhere in the code? http://codereview.chromium.org/6839015/diff/1/src/hydrogen.cc#newcode2808 src/hydrogen.cc:2808: ASSERT(!HasStackOverflow()); Turn these three checks into a single check, like ASSERT(HasNotBailedOut())? http://codereview.chromium.org/6839015/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
