[compiler-rt] c5995e2 - [sanitizer_common][test-only] Specify full path for sort executable in popen.cpp (#171622)

via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 10 10:00:03 PST 2025


Author: Dan Blackwell
Date: 2025-12-10T17:59:59Z
New Revision: c5995e25ba6a32b839cefe86abf721f57f9bf615

URL: https://github.com/llvm/llvm-project/commit/c5995e25ba6a32b839cefe86abf721f57f9bf615
DIFF: https://github.com/llvm/llvm-project/commit/c5995e25ba6a32b839cefe86abf721f57f9bf615.diff

LOG: [sanitizer_common][test-only] Specify full path for sort executable in popen.cpp (#171622)

This test has begun failing on iossim with 'sh: sort: command not found'
in the stderr. I believe this may be due to the change to the lit
internal shell not having 'sort' in it's path.

This patch adds the full path /usr/bin/sort to work around this.

Added: 
    

Modified: 
    compiler-rt/test/sanitizer_common/TestCases/Posix/popen.cpp

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/sanitizer_common/TestCases/Posix/popen.cpp b/compiler-rt/test/sanitizer_common/TestCases/Posix/popen.cpp
index 6bf6255a697a1..fdbdb2f788b04 100644
--- a/compiler-rt/test/sanitizer_common/TestCases/Posix/popen.cpp
+++ b/compiler-rt/test/sanitizer_common/TestCases/Posix/popen.cpp
@@ -8,7 +8,7 @@
 int main(void) {
   // use a tool that produces 
diff erent output than input to verify
   // that everything worked correctly
-  FILE *fp = popen("sort", "w");
+  FILE *fp = popen("/usr/bin/sort", "w");
   assert(fp);
 
   // verify that fileno() returns a meaningful descriptor (needed


        


More information about the llvm-commits mailing list