Remove old EffectAnalyzer hacks for asm.js debugInfo (#4457)

In asm2wasm we modelled debugInfo using special imports. And we tried
to not move them around much. Current debugInfo is tracked on
instructions and is not affected by removing this.

This may have some tiny effect beneficial effect on code size in debug
builds, perhaps.

1 files changed, 1 insertions, 9 deletions

diff --git a/src/ir/effects.h b/src/ir/effects.h
index 2bce174ef..2bd209d3e 100644
--- a/src/ir/effects.h
+++ b/src/ir/effects.h
@@ -31,8 +31,7 @@ public:
                  Module& module,
                  Expression* ast = nullptr)
     : ignoreImplicitTraps(passOptions.ignoreImplicitTraps),
-      trapsNeverHappen(passOptions.trapsNeverHappen),
-      debugInfo(passOptions.debugInfo), module(module),
+      trapsNeverHappen(passOptions.trapsNeverHappen), module(module),
       features(module.features) {
     if (ast) {
       walk(ast);
@@ -41,7 +40,6 @@ public:
 
   bool ignoreImplicitTraps;
   bool trapsNeverHappen;
-  bool debugInfo;
   Module& module;
   FeatureSet features;
 
@@ -433,12 +431,6 @@ private:
       if (curr->isReturn) {
         parent.branchesOut = true;
       }
-      if (parent.debugInfo) {
-        // debugInfo call imports must be preserved very strongly, do not
-        // move code around them
-        // FIXME: we could check if the call is to an import
-        parent.branchesOut = true;
-      }
     }
     void visitCallIndirect(CallIndirect* curr) {
       parent.calls = true;