LGTM!
http://codereview.chromium.org/6670119/diff/1/src/ia32/code-stubs-ia32.cc File src/ia32/code-stubs-ia32.cc (left): http://codereview.chromium.org/6670119/diff/1/src/ia32/code-stubs-ia32.cc#oldcode2830 src/ia32/code-stubs-ia32.cc:2830: masm->isolate()->cpu_features()->IsEnabled(SSE2)) { Auch. Good catch! http://codereview.chromium.org/6670119/diff/1/src/v8.cc File src/v8.cc (right): http://codereview.chromium.org/6670119/diff/1/src/v8.cc#newcode211 src/v8.cc:211: // Peephole optimization might interfere with deoptimization. This is fine. Maybe we should file a bug report on getting rid of peephole optimizations. Now that crankshaft is the default we should just rip it out. http://codereview.chromium.org/6670119/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
