LGTM.
http://codereview.chromium.org/125245/diff/6/7 File src/runtime.cc (right): http://codereview.chromium.org/125245/diff/6/7#newcode4187 Line 4187: m *= m; Can't you use m = pow(m, 2) here? http://codereview.chromium.org/125245/diff/6/7#newcode4215 Line 4215: } else if (y == 0) { I would move this test above y == +- 0.5. http://codereview.chromium.org/125245 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
