LGTM with comments
http://codereview.chromium.org/668154/diff/1/3 File src/ia32/codegen-ia32.cc (right): http://codereview.chromium.org/668154/diff/1/3#newcode6080 src/ia32/codegen-ia32.cc:6080: CpuFeatures::Scope use_cmov(CMOV); If it makes sense to use cmov here then please implement the missing variant taking a handle to avoid the additional move. http://codereview.chromium.org/668154 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
