[compiler-rt] d124ac0 - [MemProf] Temporarily disable test failing on a couple bots

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 29 11:26:49 PDT 2020


Author: Teresa Johnson
Date: 2020-10-29T11:26:21-07:00
New Revision: d124ac0c223a4d3bd537298140bb8539d6d929eb

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

LOG: [MemProf] Temporarily disable test failing on a couple bots

I finally see why this test is failing (on now 2 bots). Somehow the path
name is getting messed up, and the "linux" converted to "1". I suspect
there is something in the environment causing the macro expansion in the
test to get messed up:

http://lab.llvm.org:8011/#/builders/112/builds/555/steps/5/logs/FAIL__MemProfiler-x86_64-linux__log_path_test_cpp
http://lab.llvm.org:8011/#/builders/37/builds/275/steps/31/logs/stdio

On the avr bot:
-DPROFILE_NAME_VAR="/home/buildbot/llvm-avr-linux/llvm-avr-linux/stage1/projects/compiler-rt/test/memprof/X86_64LinuxConfig/TestCases/Output/log_path_test.cpp.tmp.log2"

after macros expansions becomes:
/home/buildbot/llvm-avr-1/llvm-avr-1/stage1/projects/compiler-rt/test/memprof/X86_64LinuxConfig/TestCases/Output/log_path_test.cpp.tmp.log2

Similar (s/linux/1/) on the other bot.

Disable it while I investigate

Added: 
    

Modified: 
    compiler-rt/test/memprof/TestCases/log_path_test.cpp

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/memprof/TestCases/log_path_test.cpp b/compiler-rt/test/memprof/TestCases/log_path_test.cpp
index 88a8fac33950..92d194be707e 100644
--- a/compiler-rt/test/memprof/TestCases/log_path_test.cpp
+++ b/compiler-rt/test/memprof/TestCases/log_path_test.cpp
@@ -19,12 +19,11 @@
 // RUN:   not %run %t 2>&1 | FileCheck %s --check-prefix=CHECK-LONG --dump-input=always
 
 // Specifying the log name via the __memprof_profile_filename variable.
-// RUN: %clangxx_memprof  %s -o %t -DPROFILE_NAME_VAR="%t.log2"
-// RUN: ls -l %T | FileCheck %s --dump-input=always
-// RUN: rm -f %t.log2.*
-// RUN: ls -l %T | FileCheck %s --dump-input=always
-// RUN: %run %t
-// RUN: FileCheck %s --check-prefix=CHECK-GOOD --dump-input=always < %t.log2.*
+// Temporarily disable until bot failures fixed
+// %clangxx_memprof  %s -o %t -DPROFILE_NAME_VAR="%t.log2"
+// rm -f %t.log2.*
+// %run %t
+// FileCheck %s --check-prefix=CHECK-GOOD --dump-input=always < %t.log2.*
 
 #ifdef PROFILE_NAME_VAR
 #define xstr(s) str(s)
@@ -43,4 +42,3 @@ int main(int argc, char **argv) {
 // CHECK-GOOD: Memory allocation stack id
 // CHECK-INVALID: ERROR: Can't open file: /dev/null/INVALID
 // CHECK-LONG: ERROR: Path is too long: 01234
-// CHECK: log_path_test


        


More information about the llvm-commits mailing list