[libcxx-commits] [compiler-rt] [libcxx] [libcxxabi] [libunwind] [libunwind][libcxx][libcxxabi][compiler-rt-builtins] Fix Exception Handling build for wasm (PR #79667)
via libcxx-commits
libcxx-commits at lists.llvm.org
Mon Apr 1 09:04:54 PDT 2024
================
@@ -12,6 +12,7 @@
#ifndef __LIBUNWIND_EXT__
#define __LIBUNWIND_EXT__
+#ifndef __wasm__
----------------
EricWF wrote:
I would much rather you guard the usage of this header in each file.
That way it's clear to the reader in context that the include is unused under a certain configuration.
If the entire file contains no WASM code, then it probably shouldn't mention WASM at all.
Can you please explain what the `__WASM_EXCEPTIONS__` or `__USING_WASM_EXCEPTIONS__` was?
https://github.com/llvm/llvm-project/pull/79667
More information about the libcxx-commits
mailing list