Skip to content

Conversation

@PiJoules
Copy link
Contributor

The declaration was static when it shouldn't be since it can be defined in FuzzerUtilFuchsia.cpp

The declaration was static when it shouldn't be.
@llvmbot
Copy link
Member

llvmbot commented Aug 26, 2025

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

Author: None (PiJoules)

Changes

The declaration was static when it shouldn't be since it can be defined in FuzzerUtilFuchsia.cpp


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

1 Files Affected:

  • (modified) compiler-rt/lib/fuzzer/FuzzerDriver.cpp (+2-2)
diff --git a/compiler-rt/lib/fuzzer/FuzzerDriver.cpp b/compiler-rt/lib/fuzzer/FuzzerDriver.cpp
index cd143f1089ef0..af9c260537e2f 100644
--- a/compiler-rt/lib/fuzzer/FuzzerDriver.cpp
+++ b/compiler-rt/lib/fuzzer/FuzzerDriver.cpp
@@ -306,7 +306,7 @@ static int RunInMultipleProcesses(const std::vector<std::string> &Args,
   return HasErrors ? 1 : 0;
 }
 
-static void StartRssThread(Fuzzer *F, size_t RssLimitMb);
+void StartRssThread(Fuzzer *F, size_t RssLimitMb);
 
 // Fuchsia needs to do some book checking before starting the RssThread,
 // so it has its own implementation.
@@ -320,7 +320,7 @@ static void RssThread(Fuzzer *F, size_t RssLimitMb) {
   }
 }
 
-static void StartRssThread(Fuzzer *F, size_t RssLimitMb) {
+void StartRssThread(Fuzzer *F, size_t RssLimitMb) {
   if (!RssLimitMb)
     return;
   std::thread T(RssThread, F, RssLimitMb);

@PiJoules PiJoules merged commit 3160047 into llvm:main Aug 26, 2025
13 checks passed
@PiJoules PiJoules deleted the fuzzer-fix branch August 26, 2025 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants