From 9c75b8f8ef58347f2fab44463f9a65eb37212742 Mon Sep 17 00:00:00 2001
From: Thomas Lively <7121787+tlively@users.noreply.github.com>
Date: Thu, 9 Apr 2020 11:41:40 -0700
Subject: Remove function index printing (#2742)

`BinaryIndexes` was only used in two places (Print.cpp and
wasm-binary.h), so it didn't seem to be a great fit for
module-utils.h. This change moves it to wasm-binary.h and removes its
usage in Print.cpp. This means that function indexes are no longer
printed, but those were of limited utility and were the source of
annoying noise when updating tests, anyway.
---
 src/ir/module-utils.h | 52 ---------------------------------------------------
 1 file changed, 52 deletions(-)

(limited to 'src/ir/module-utils.h')

diff --git a/src/ir/module-utils.h b/src/ir/module-utils.h
index 485bc4833..e589f5052 100644
--- a/src/ir/module-utils.h
+++ b/src/ir/module-utils.h
@@ -28,58 +28,6 @@ namespace wasm {
 
 namespace ModuleUtils {
 
-// Computes the indexes in a wasm binary, i.e., with function imports
-// and function implementations sharing a single index space, etc.,
-// and with the imports first (the Module's functions and globals
-// arrays are not assumed to be in a particular order, so we can't
-// just use them directly).
-struct BinaryIndexes {
-  std::unordered_map<Name, Index> functionIndexes;
-  std::unordered_map<Name, Index> eventIndexes;
-  std::unordered_map<Name, Index> globalIndexes;
-
-  BinaryIndexes(Module& wasm) {
-    auto addIndexes = [&](auto& source, auto& indexes) {
-      auto addIndex = [&](auto* curr) {
-        auto index = indexes.size();
-        indexes[curr->name] = index;
-      };
-      for (auto& curr : source) {
-        if (curr->imported()) {
-          addIndex(curr.get());
-        }
-      }
-      for (auto& curr : source) {
-        if (!curr->imported()) {
-          addIndex(curr.get());
-        }
-      }
-    };
-    addIndexes(wasm.functions, functionIndexes);
-    addIndexes(wasm.events, eventIndexes);
-
-    // Globals may have tuple types in the IR, in which case they lower to
-    // multiple globals, one for each tuple element, in the binary. Tuple
-    // globals therefore occupy multiple binary indices, and we have to take
-    // that into account when calculating indices.
-    Index globalCount = 0;
-    auto addGlobal = [&](auto* curr) {
-      globalIndexes[curr->name] = globalCount;
-      globalCount += curr->type.size();
-    };
-    for (auto& curr : wasm.globals) {
-      if (curr->imported()) {
-        addGlobal(curr.get());
-      }
-    }
-    for (auto& curr : wasm.globals) {
-      if (!curr->imported()) {
-        addGlobal(curr.get());
-      }
-    }
-  }
-};
-
 inline Function* copyFunction(Function* func, Module& out) {
   auto* ret = new Function();
   ret->name = func->name;
-- 
cgit v1.2.3