Add no-opt test.

emscripten-core · Sep 11, 2024 · 06dc320 · 06dc320
1 parent 5e37239
commit 06dc320
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/test/test_other.py b/test/test_other.py
@@ -15143,11 +15143,13 @@ def test_fs_writev_partial_write(self):
     self.do_run_in_out_file_test('fs/test_writev_partial_write.c')
 
   @requires_v8
-  def test_fp16(self):
+  @parameterized({
+    '': [[]],
+    'O2': [['-O2']]
+  })
+  def test_fp16(self, opts):
     self.v8_args += ['--experimental-wasm-fp16']
     # TODO Remove this. Liftoff is currently broken for this test.
     self.v8_args += ['--no-liftoff']
-    # TODO Test this in a different optimization mode. Only O2 currently works.
-    self.emcc_args = ['-msimd128', '-mfp16', '-sENVIRONMENT=shell']
-    # self.emcc_args += [ '-O2']
+    self.emcc_args = ['-msimd128', '-mfp16', '-sENVIRONMENT=shell'] + opts
     self.do_runf('test_fp16.c')