LGTM modulo restoring an ASSERT (see in comments). Thanks a lot for refactoring this---it's notably better than it was before.
http://codereview.chromium.org/2251003/diff/12001/13001 File src/arm/stub-cache-arm.cc (right): http://codereview.chromium.org/2251003/diff/12001/13001#newcode924 src/arm/stub-cache-arm.cc:924: } Please, restore ASSERT added in http://codereview.chromium.org/2279005/show http://codereview.chromium.org/2251003/diff/12001/13002 File src/ia32/stub-cache-ia32.cc (right): http://codereview.chromium.org/2251003/diff/12001/13002#newcode956 src/ia32/stub-cache-ia32.cc:956: } ditto for ia32 and x64 http://codereview.chromium.org/2251003/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
