Merge pull request #695 from WebAssembly/opts

Get optimizer on par with emscripten asm.js optimizer

1 files changed, 10 insertions, 0 deletions

diff --git a/src/passes/DuplicateFunctionElimination.cpp b/src/passes/DuplicateFunctionElimination.cpp
index 2b8e69b54..cfe2d8565 100644
--- a/src/passes/DuplicateFunctionElimination.cpp
+++ b/src/passes/DuplicateFunctionElimination.cpp
@@ -102,7 +102,17 @@ struct DuplicateFunctionElimination : public Pass {
         auto& group = pair.second;
         if (group.size() == 1) continue;
         // pick a base for each group, and try to replace everyone else to it. TODO: multiple bases per hash group, for collisions
+#if 0
+        // for comparison purposes, pick in a deterministic way based on the names
+        Function* base = nullptr;
+        for (auto* func : group) {
+          if (!base || strcmp(func->name.str, base->name.str) < 0) {
+            base = func;
+          }
+        }
+#else
         Function* base = group[0];
+#endif
         for (auto* func : group) {
           if (func != base && equal(func, base)) {
             replacements[func->name] = base->name;