LGTM with a tiny nit.
http://codereview.chromium.org/9690010/diff/5001/src/ia32/codegen-ia32.cc File src/ia32/codegen-ia32.cc (right): http://codereview.chromium.org/9690010/diff/5001/src/ia32/codegen-ia32.cc#newcode120 src/ia32/codegen-ia32.cc:120: ASSERT(CpuFeatures::IsSupported(SSE2)); I think we can safely remove this assertion. http://codereview.chromium.org/9690010/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
