Changeset 56135 in webkit for trunk/JavaScriptCore/runtime/MathObject.cpp

Timestamp:

Mar 17, 2010, 5:10:07 PM (15 years ago)

Author:

Darin Adler

Message:

Speed up Math.round a little by removing unneeded special case
​https://bugs.webkit.org/show_bug.cgi?id=36107

Reviewed by Oliver Hunt.

Test: fast/js/math.html

• runtime/MathObject.cpp:

(JSC::mathProtoFuncRound): This function had a special case for numbers
between -0.5 and -0.0 to return -0.0. But the algorithm in the function
already yields -0.0 for those cases, so the extra checking and branching
is unneeded.

File:

• trunk/JavaScriptCore/runtime/MathObject.cpp (modified) (1 diff)

trunk/JavaScriptCore/runtime/MathObject.cpp

@@ -217 +217 @@
 {
     double arg = args.at(0).toNumber(exec);
-    if (signbit(arg) && arg >= -0.5)
-         return jsNumber(exec, -0.0);
     double integer = ceil(arg);
     return jsNumber(exec, integer - (integer - arg > 0.5));