LGTM
http://codereview.chromium.org/521019/diff/1/5 File src/arm/fast-codegen-arm.cc (right): http://codereview.chromium.org/521019/diff/1/5#newcode255 src/arm/fast-codegen-arm.cc:255: case Slot::PARAMETER: // Fall through. I think two cases immediately after each other is a known idion, and doesn't need the "Fall through" comment. I would reserve that for falling through a case that actually does something. http://codereview.chromium.org/521019 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
