LGTM with suggestion.
http://codereview.chromium.org/10967064/diff/1/test/mjsunit/math-floor-part4.js File test/mjsunit/math-floor-part4.js (right): http://codereview.chromium.org/10967064/diff/1/test/mjsunit/math-floor-part4.js#newcode76 test/mjsunit/math-floor-part4.js:76: assertEquals(-0, floorsum(100000, -0)); It seems that this part has been added in r8877 as a regression test. Maybe you want to make this a separate file to make it stand apart? http://codereview.chromium.org/10967064/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
