LGTM.
http://codereview.chromium.org/6701003/diff/2001/src/x64/lithium-codegen-x64.cc File src/x64/lithium-codegen-x64.cc (right): http://codereview.chromium.org/6701003/diff/2001/src/x64/lithium-codegen-x64.cc#newcode873 src/x64/lithium-codegen-x64.cc:873: } else if (right_value == 0) { If BailoutOnMinusZero is true, we always bail out if a constant right_value is 0, after doing a lot of work. Maybe we should only bail out if the left value is negative, and do so immediately, in special case code. http://codereview.chromium.org/6701003/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
