From 4b05489435a8f7c4a149db16a11f6c82ce63d622 Mon Sep 17 00:00:00 2001
From: Heejin Ahn <aheejin@gmail.com>
Date: Tue, 28 May 2019 15:55:32 -0700
Subject: Simplify function lists in RemoveUnusedModuleElements (NFC) (#2147)

It looks there's no need to maintain `functions` and `functionImports`
separately.
---
 src/passes/RemoveUnusedModuleElements.cpp | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

(limited to 'src')

diff --git a/src/passes/RemoveUnusedModuleElements.cpp b/src/passes/RemoveUnusedModuleElements.cpp
index ec9753367..c96ce1a8c 100644
--- a/src/passes/RemoveUnusedModuleElements.cpp
+++ b/src/passes/RemoveUnusedModuleElements.cpp
@@ -120,17 +120,12 @@ struct ReachabilityAnalyzer : public PostWalker<ReachabilityAnalyzer> {
 // Finds function type usage
 
 struct FunctionTypeAnalyzer : public PostWalker<FunctionTypeAnalyzer> {
-  std::vector<Function*> functionImports;
   std::vector<Function*> functions;
   std::vector<CallIndirect*> indirectCalls;
 
   void visitFunction(Function* curr) {
     if (curr->type.is()) {
-      if (curr->imported()) {
-        functionImports.push_back(curr);
-      } else {
-        functions.push_back(curr);
-      }
+      functions.push_back(curr);
     }
   }
 
@@ -271,9 +266,6 @@ struct RemoveUnusedModuleElements : public Pass {
       }
     };
     // canonicalize all uses of function types
-    for (auto* import : analyzer.functionImports) {
-      import->type = canonicalize(import->type);
-    }
     for (auto* func : analyzer.functions) {
       func->type = canonicalize(func->type);
     }
-- 
cgit v1.2.3