Unified module loader support in binaryen.js (#1074)

* Unified module loader support in binaryen.js

* Recompiled binaryen.js and wasm.js

Author: dcodeIO

bin/binaryen.js

@@ -71,6 +71,7 @@ echo "building wasm.js"
   src/passes/ExtractFunction.cpp \
   src/passes/FlattenControlFlow.cpp \
   src/passes/Inlining.cpp \
+  src/passes/InstrumentLocals.cpp \
   src/passes/InstrumentMemory.cpp \
   src/passes/LegalizeJSInterface.cpp \
   src/passes/LocalCSE.cpp \
@@ -95,7 +96,9 @@ echo "building wasm.js"
   src/passes/ReorderFunctions.cpp \
   src/passes/ReorderLocals.cpp \
   src/passes/ReReloop.cpp \
+  src/passes/SSAify.cpp \
   src/passes/SimplifyLocals.cpp \
+  src/passes/Untee.cpp \
   src/passes/Vacuum.cpp \
   src/emscripten-optimizer/parser.cpp \
   src/emscripten-optimizer/simple_ast.cpp \
@@ -139,6 +142,7 @@ fi
   src/passes/ExtractFunction.cpp \
   src/passes/FlattenControlFlow.cpp \
   src/passes/Inlining.cpp \
+  src/passes/InstrumentLocals.cpp \
   src/passes/InstrumentMemory.cpp \
   src/passes/LegalizeJSInterface.cpp \
   src/passes/LocalCSE.cpp \
@@ -163,7 +167,9 @@ fi
   src/passes/ReorderFunctions.cpp \
   src/passes/ReorderLocals.cpp \
   src/passes/ReReloop.cpp \
+  src/passes/SSAify.cpp \
   src/passes/SimplifyLocals.cpp \
+  src/passes/Untee.cpp \
   src/passes/Vacuum.cpp \
   src/wasm-emscripten.cpp \
   src/support/colors.cpp \

bin/wasm.js

@@ -511,7 +511,7 @@ def fix(x):
 print '\n[ checking example testcases... ]\n'
 
 if options.run_gcc_tests:
-  print '\n[ checking native gcc testcases...]\n'  
+  print '\n[ checking native gcc testcases...]\n'
   if not NATIVECC or not NATIVEXX:
     fail_with_error('Native compiler (e.g. gcc/g++) was not found in PATH!')
   else:

build-js.sh

@@ -911,34 +911,19 @@
     return Module['_BinaryenSetAPITracing'](on);
   };
 
-  return Module;
-};
+  // Support AMD-compatible loaders by defining a factory function that returns 'Module'
+  if (typeof define === "function" && define["amd"])
+    define(function() { return Module; });
 
-if (typeof exports != 'undefined') {
-  (function(){
-    var a = Binaryen();
-    if (typeof module === 'object') {
-      module.exports = a;
-    } else {
-      for (var k in a) {
-        exports[k] = a[k];
-      }
-    }
-  })();
-}
-(typeof window !== 'undefined' ? window :
- typeof global !== 'undefined' && (
-  typeof process === 'undefined' ||
+  // Support CommonJS-compatible loaders by checking for 'require' and 'module.exports'
+  else if (typeof require === "function" && typeof module !== "undefined" && module && module.exports)
+    module.exports = Module;
 
-  // Note: We must export "Binaryen" even inside a CommonJS/AMD/UMD module
-  // space because check.py generates a.js which requires Binaryen global var
-  ( process.argv &&
-    Array.isArray(process.argv) &&
-    process.argv[1] &&
-    (process.argv[1].substr(-5) === '/a.js' ||
-     process.argv[1].substr(-5) === '\\a.js')
-  )
+  // Otherwise expose as 'Binaryen' globally checking for common names of the global object
+  // first (namely 'global' and 'window') and fall back to 'this' (i.e. within web workers).
+  else
+    (typeof global !== "undefined" && global ||
+     typeof window !== "undefined" && window ||
+     this)["Binaryen"] = Module;
 
- ) ? global :
- this
-)['Binaryen'] = Binaryen();
+})();

check.py

@@ -1 +1,2 @@
-var Binaryen = function(Module) {
+(function() {
+  "use strict";