[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
================
@@ -15,6 +15,7 @@
#ifndef UNWIND_ASSEMBLY_H
#define UNWIND_ASSEMBLY_H
+#ifndef __wasm__
----------------
EricWF wrote:
Again, this is not OK.
First, this file just defines macros, so even under WASM it should be safe to include.
If you're hitting the `#error` case in the `#else`, then add an `#elif __wasm__`.
While his style of code may not cause issues for you, because you don't use it. It makes things more complicated for every change that actually uses these files.
https://github.com/llvm/llvm-project/pull/79667
More information about the libcxx-commits
mailing list