Bug 881882 - OdinMonkey: take the percentage back out (it's wrong and misleading with parallel compilation) (r=me)
authorLuke Wagner <luke@mozilla.com>
Fri, 14 Jun 2013 10:09:18 -0700
changeset 135459 5527906f11f5edd0e6bf3320816c31e01655729e
parent 135458 d22a6e206de0c2b54309f6c6fff226920afabd6c
child 135460 1f7059b5a6c6c53447ef95d25e71a87b0c444dd2
push id270
push userpvanderbeken@mozilla.com
push dateThu, 06 Mar 2014 09:24:21 +0000
reviewersme
bugs881882
milestone24.0a1
Bug 881882 - OdinMonkey: take the percentage back out (it's wrong and misleading with parallel compilation) (r=me)
js/src/ion/AsmJS.cpp
--- a/js/src/ion/AsmJS.cpp
+++ b/js/src/ion/AsmJS.cpp
@@ -1496,19 +1496,18 @@ class MOZ_STACK_CLASS ModuleCompiler
             slowFuns.reset(JS_smprintf("; %d functions compiled slowly: ", slowFunctions_.length()));
             if (!slowFuns)
                 return;
             for (unsigned i = 0; i < slowFunctions_.length(); i++) {
                 SlowFunction &func = slowFunctions_[i];
                 JSAutoByteString name;
                 if (!js_AtomToPrintableString(cx_, func.name, &name))
                     return;
-                slowFuns.reset(JS_smprintf("%s%s:%u:%u (%ums, %g%%)%s", slowFuns.get(),
+                slowFuns.reset(JS_smprintf("%s%s:%u:%u (%ums)%s", slowFuns.get(),
                                            name.ptr(), func.line, func.column, func.ms,
-                                           double(func.ms)/double(msTotal),
                                            i+1 < slowFunctions_.length() ? ", " : ""));
                 if (!slowFuns)
                     return;
             }
         }
         out->reset(JS_smprintf("total compilation time %dms%s",
                                msTotal, slowFuns ? slowFuns.get() : ""));
     }