[ASan] Recognize WASI platform in sanitizer_platform.h

[kateinoigakukun]

We are working porting ASan to Wasm/WASI (not Emscripten). This is the groundwork for upcoming WASI support in ASan runtime library.

[llvmbot]

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Yuta Saito (kateinoigakukun)

Changes

The groundwork for upcoming WASI support.

---

Full diff: https://github.com/llvm/llvm-project/pull/139017.diff

1 Files Affected:

• (modified) compiler-rt/lib/sanitizer_common/sanitizer_platform.h (+8-1)

diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_platform.h b/compiler-rt/lib/sanitizer_common/sanitizer_platform.h
index 9f5f41cd85514..0a116da0d895b 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_platform.h
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_platform.h
@@ -14,7 +14,8 @@
 
 #if !defined(__linux__) && !defined(__FreeBSD__) && !defined(__NetBSD__) && \
     !defined(__APPLE__) && !defined(_WIN32) && !defined(__Fuchsia__) &&     \
-    !(defined(__sun__) && defined(__svr4__)) && !defined(__HAIKU__)
+    !(defined(__sun__) && defined(__svr4__)) && !defined(__HAIKU__) && \
+    !defined(__wasi__)
 #  error "This operating system is not supported"
 #endif
 
@@ -61,6 +62,12 @@
 #  define SANITIZER_HAIKU 0
 #endif
 
+#if defined(__wasi__)
+#  define SANITIZER_WASI 1
+#else
+#  define SANITIZER_WASI 0
+#endif
+
 // - SANITIZER_APPLE: all Apple code
 //   - TARGET_OS_OSX: macOS
 //   - SANITIZER_IOS: devices (iOS and iOS-like)

[github-actions]

✅ With the latest revision this PR passed the C/C++ code formatter.

[dschuff]

/cc @sunfishcode @sbc100
That reminds me we should think about upstreaming our Emscripten sanitizer implementation.
@kateinoigakukun there may not be a huge amount of overlap with WASI since I suspect that the "OS" layers are more relevant here than the architecture, but it might be worth taking a look at our version here

[kateinoigakukun]

Yeah, I think we can share a few things about shadow memory mapping. If you're happy, I can upstream a part of the Emscripten stuff at the same time later.

[dschuff]

Sounds good to me (also /cc @aheejin).

[kateinoigakukun]

While this is a very platform-scoped change, I'd like to have explicit approval before landing. Would someone give approval on this?

[kateinoigakukun]

Thanks!