[compiler-rt] [sanitizer] Downgrade TestPTrace() Reports to VReport (PR #152350)

via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 6 11:05:51 PDT 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

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

Author: Thurston Dang (thurstond)

<details>
<summary>Changes</summary>

Requested in https://github.com/llvm/llvm-project/pull/152072#discussion_r2257892739

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


1 Files Affected:

- (modified) compiler-rt/lib/sanitizer_common/sanitizer_stoptheworld_linux_libcdep.cpp (+7-6) 


``````````diff
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_stoptheworld_linux_libcdep.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_stoptheworld_linux_libcdep.cpp
index 5fde65ea48c42..2bf547f4a7216 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_stoptheworld_linux_libcdep.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_stoptheworld_linux_libcdep.cpp
@@ -413,11 +413,12 @@ static void TestPTrace() {
   // internal_fork() on SPARC actually calls __fork(). We can't safely fork,
   // because it's possible seccomp has been configured to disallow fork() but
   // allow clone().
-  Report("WARNING: skipping TestPTrace() because this is SPARC\n");
-  Report(
-      "If seccomp blocks ptrace, LeakSanitizer may hang without further "
-      "notice\n");
-  Report(
+  VReport(1, "WARNING: skipping TestPTrace() because this is SPARC\n");
+  VReport(1,
+          "If seccomp blocks ptrace, LeakSanitizer may hang without further "
+          "notice\n");
+  VReport(
+      1,
       "If seccomp does not block ptrace, you can safely ignore this warning\n");
 #  else
   // Heuristic: only check the first time this is called. This is not always
@@ -438,7 +439,7 @@ static void TestPTrace() {
   if (pid < 0) {
     int rverrno;
     if (internal_iserror(pid, &rverrno))
-      Report("WARNING: TestPTrace() failed to fork (errno %d)\n", rverrno);
+      VReport(0, "WARNING: TestPTrace() failed to fork (errno %d)\n", rverrno);
 
     // We don't abort the sanitizer - it's still worth letting the sanitizer
     // try.

``````````

</details>


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


More information about the llvm-commits mailing list