Removed redundant wrapping function around Runtime.stackAlloc, Runtime.stackSave and Runtime.stackRestore assignments to their respective versions in the asm namespace. Reran test suite and retested with my own app which makes heavy use of stackSave and stackRestore.

Author: fadams

emscripten.py

@@ -648,9 +648,9 @@ def math_fix(g):
 
     if not settings.get('SIDE_MODULE'):
       funcs_js.append('''
-Runtime.stackAlloc = function(size) { return asm['stackAlloc'](size) };
-Runtime.stackSave = function() { return asm['stackSave']() };
-Runtime.stackRestore = function(top) { asm['stackRestore'](top) };
+Runtime.stackAlloc = asm['stackAlloc'];
+Runtime.stackSave = asm['stackSave'];
+Runtime.stackRestore = asm['stackRestore'];
 Runtime.getTempRet0 = asm['getTempRet0'];
 ''')
 
@@ -1229,9 +1229,9 @@ def math_fix(g):
 
     if not settings.get('SIDE_MODULE'):
       funcs_js.append('''
-Runtime.stackAlloc = function(size) { return asm['stackAlloc'](size) };
-Runtime.stackSave = function() { return asm['stackSave']() };
-Runtime.stackRestore = function(top) { asm['stackRestore'](top) };
+Runtime.stackAlloc = asm['stackAlloc'];
+Runtime.stackSave = asm['stackSave'];
+Runtime.stackRestore = asm['stackRestore'];
 Runtime.getTempRet0 = asm['getTempRet0'];
 ''')