[compiler-rt] [LoongArch][sanitizer] Fix SC_ADDRERR_{RD,WR} missing in the musl environment (PR #108557)

via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 13 06:01:43 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

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

Author: hev (heiher)

<details>
<summary>Changes</summary>

Fixes #<!-- -->108550

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


1 Files Affected:

- (modified) compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp (+12) 


``````````diff
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
index 16d06c69a38f32..f4a2f46c6ea022 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
@@ -2018,6 +2018,18 @@ SignalContext::WriteFlag SignalContext::GetWriteFlag() const {
     return Unknown;
   return esr & ESR_ELx_WNR ? Write : Read;
 #  elif defined(__loongarch__)
+     // In the musl environment, the Linux kernel uapi sigcontext.h is not
+     // included in signal.h. To avoid missing the SC_ADDRERR_{RD,WR} macros,
+     // copy them here. The LoongArch Linux kernel uapi is already stable,
+     // so there's no need to worry about the value changing.
+#    ifndef SC_ADDRERR_RD
+       // Address error was due to memory load
+#      define SC_ADDRERR_RD		(1 << 30)
+#    endif
+#    ifndef SC_ADDRERR_WR
+       // Address error was due to memory store
+#      define SC_ADDRERR_WR		(1 << 31)
+#    endif
   u32 flags = ucontext->uc_mcontext.__flags;
   if (flags & SC_ADDRERR_RD)
     return SignalContext::Read;

``````````

</details>


https://github.com/llvm/llvm-project/pull/108557


More information about the llvm-commits mailing list