Skip to content

Commit

Permalink
Enable sse tests under wasm64. NFC (emscripten-core#21051)
Browse files Browse the repository at this point in the history
  • Loading branch information
sbc100 committed Jan 10, 2024
1 parent 20865d7 commit a997567
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions test/test_core.py
Original file line number Diff line number Diff line change
Expand Up @@ -6521,12 +6521,13 @@ def test_sse1(self):
# Tests invoking the SIMD API via x86 SSE2 emmintrin.h header (_mm_x() functions)
@wasm_simd
@requires_native_clang
@no_wasm64('https://github.com/llvm/llvm-project/issues/57577')
@no_safe_heap('has unaligned 64-bit operations in wasm')
@is_slow_test
@no_ubsan('https://github.com/emscripten-core/emscripten/issues/19688')
@no_asan('local count too large')
def test_sse2(self):
if self.is_wasm64():
self.require_node_canary()
src = test_file('sse/test_sse2.cpp')
self.run_process([shared.CLANG_CXX, src, '-msse2', '-Wno-argument-outside-range', '-o', 'test_sse2', '-D_CRT_SECURE_NO_WARNINGS=1'] + clang_native.get_clang_native_args(), stdout=PIPE)
native_result = self.run_process('./test_sse2', stdout=PIPE).stdout
Expand All @@ -6548,7 +6549,6 @@ def test_sse3(self):
self.do_runf(src, native_result)

# Tests invoking the SIMD API via x86 SSSE3 tmmintrin.h header (_mm_x() functions)
@no_wasm64('https://github.com/llvm/llvm-project/issues/57577')
@wasm_simd
@requires_native_clang
def test_ssse3(self):
Expand All @@ -6561,12 +6561,13 @@ def test_ssse3(self):
self.do_runf(src, native_result)

# Tests invoking the SIMD API via x86 SSE4.1 smmintrin.h header (_mm_x() functions)
@no_wasm64('https://github.com/llvm/llvm-project/issues/57577')
@no_ubsan('https://github.com/emscripten-core/emscripten/issues/19749')
@wasm_simd
@requires_native_clang
@is_slow_test
def test_sse4_1(self):
if self.is_wasm64():
self.require_node_canary()
src = test_file('sse/test_sse4_1.cpp')
if not self.is_optimizing() and '-fsanitize=address' in self.emcc_args:
# ASan with -O0 fails with:
Expand Down

0 comments on commit a997567

Please sign in to comment.