LGTM
http://codereview.chromium.org/604054/diff/1/2 File src/arm/stub-cache-arm.cc (right): http://codereview.chromium.org/604054/diff/1/2#newcode1055 src/arm/stub-cache-arm.cc:1055: <<<<<<< .mine merge conflict http://codereview.chromium.org/604054/diff/1/2#newcode1065 src/arm/stub-cache-arm.cc:1065: CallInterceptorCompiler compiler(arguments()); 1) probably the whole compiler should go away as it now only have one method; 2) I am slightly concerned that various platform now diverge---could we bring the similar change to other platforms? http://codereview.chromium.org/604054 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
