disable closure in some non-asm tests where it fails to properly minify Math.imul

1 files changed, 2 insertions, 2 deletions

diff --git a/tests/runner.py b/tests/runner.py
index dba69e4e..7eacd059 100755
--- a/tests/runner.py
+++ b/tests/runner.py
@@ -7974,7 +7974,7 @@ def process(filename):
       if Settings.ASM_JS:
         self.banned_js_engines = [NODE_JS] # TODO investigate
 
-      if self.emcc_args is not None and '-O2' in self.emcc_args:
+      if self.emcc_args is not None and '-O2' in self.emcc_args and 'ASM_JS=0' not in self.emcc_args: # without asm, closure minifies Math.imul badly
         self.emcc_args += ['--closure', '1'] # Use closure here for some additional coverage
 
       Settings.CORRECT_SIGNS = 1
@@ -8697,7 +8697,7 @@ def process(filename):
 '''
         # XXX disable due to possible v8 bug -- self.do_run(src, '*166*\n*ok*', post_build=post)
 
-        if self.emcc_args is not None and '-O2' in self.emcc_args:
+        if self.emcc_args is not None and '-O2' in self.emcc_args and 'ASM_JS=0' not in self.emcc_args: # without asm, closure minifies Math.imul badly
           self.emcc_args += ['--closure', '1'] # Use closure here, to test we export things right
 
         # Way 2: use CppHeaderParser