[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:05:05 PDT 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 76fd69be74e3a52cec77d5401003469791ef9834 8f9d73bb173d382fc0fbcf36b5275acf3e396797 --extensions cpp -- compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
index f4a2f46c6e..6359f4348e 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
@@ -2018,17 +2018,17 @@ 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.
+  // 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)
+  // 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)
+  // 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)

``````````

</details>


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


More information about the llvm-commits mailing list