[compiler-rt] 4fee8a1 - [NFC][compiler-rt][profile] Remove non-Posix -h option from test

Jinsong Ji via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 14 08:08:53 PDT 2021


Author: Jinsong Ji
Date: 2021-10-14T15:08:38Z
New Revision: 4fee8a16915ec461002d1cc8e452be1e06cab6d7

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

LOG: [NFC][compiler-rt][profile] Remove non-Posix -h option from test

We are running `ls -lh` in gcov-execlp.c test in Posix folder.
However `-h` is not a POSIX option,ls on some POSIX system (eg: AIX)
may not support it.

This patch remove this option to avoid break.

Reviewed By: anhtuyen

Differential Revision: https://reviews.llvm.org/D111807

Added: 
    

Modified: 
    compiler-rt/test/profile/Posix/gcov-execlp.c

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/profile/Posix/gcov-execlp.c b/compiler-rt/test/profile/Posix/gcov-execlp.c
index 0b70e9c697904..362f59516d217 100644
--- a/compiler-rt/test/profile/Posix/gcov-execlp.c
+++ b/compiler-rt/test/profile/Posix/gcov-execlp.c
@@ -18,10 +18,10 @@ void func2(void) {}                // CHECK-NEXT: #####: [[#@LINE]]:
 int main(void) {                   // CHECK-NEXT:     1: [[#@LINE]]:
   func1();                         // CHECK-NEXT:     1: [[#@LINE]]:
 #ifdef EXECVP
-  char *argv[] = {"ls", "-l", "-h", (char *)0};
+  char *argv[] = {"ls", "-l", (char *)0};
   execvp("ls", argv);              // EXECVP:         1: [[#@LINE]]:  execvp
 #else
-  execlp("ls", "-l", "-h", (char *)0); // EXECLP:     1: [[#@LINE]]:  execlp
+  execlp("ls", "-l", (char *)0); // EXECLP:     1: [[#@LINE]]:  execlp
 #endif
   func2();                         // CHECK:      #####: [[#@LINE]]:  func2
   return 0;                        // CHECK-NEXT: #####: [[#@LINE]]:


        


More information about the llvm-commits mailing list