Eliminate [[clang::musttail]] from wasm builds.
The roll is currently failing because wasm builds are failing with
"error: WebAssembly 'tail-call' feature not enabled"
It looks like tail-calling in wasm was only added in 2023:
https://v8.dev/blog/wasm-tail-call
Tail-calls aren't critical for skcms, so let's disable it.
Change-Id: Iccdde81117e8cb5ebb7c76fd9a26869b364efae6
Reviewed-on: https://skia-review.googlesource.com/c/skcms/+/782436
Reviewed-by: Brian Osman <brianosman@google.com>
Auto-Submit: John Stiles <johnstiles@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
diff --git a/src/skcms_internals.h b/src/skcms_internals.h
index 05c1550..917cf07 100644
--- a/src/skcms_internals.h
+++ b/src/skcms_internals.h
@@ -27,9 +27,11 @@
#endif
#ifndef SKCMS_MUSTTAIL
+ // Sanitizers do not work well with [[clang::musttail]], and corrupt src/dst pointers.
+ // Tail-calls are a fairly new, optional feature in wasm and won't work everywhere.
#if __has_cpp_attribute(clang::musttail) && !__has_feature(memory_sanitizer) \
- && !__has_feature(address_sanitizer)
- // Sanitizers do not work well with [[clang::musttail]], and corrupt src/dst pointers.
+ && !__has_feature(address_sanitizer) \
+ && !defined(__EMSCRIPTEN_major__)
#define SKCMS_MUSTTAIL [[clang::musttail]]
#else
#define SKCMS_MUSTTAIL
