[clang-tools-extra] r273941 - clang-rename: try to make ClassTestReplacements more reliable

Miklos Vajna via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 27 14:04:53 PDT 2016


Author: vmiklos
Date: Mon Jun 27 16:04:53 2016
New Revision: 273941

URL: http://llvm.org/viewvc/llvm-project?rev=273941&view=rev
Log:
clang-rename: try to make ClassTestReplacements more reliable

As it failed on e.g.
<http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/7089/steps/ninja%20check%201/logs/FAIL%3A%20Clang%20Tools%3A%3AClassTestReplacements.cpp> with:

Trouble iterating over directory '/home/buildbots/ppc64be-clang-test/clang-ppc64be/stage1/tools/clang/tools/extra/test/clang-rename/Output': No such file or directory

A reliable way to trigger the problem locally is to run all clang-rename
tests in parallel in a loop:

for i in $(seq 1 100); do ~/git/llvm/workdir/bin/llvm-lit -v -j15 . || break; done

Change the test script to be more similar to test/Tooling/clang-check.cpp, that
way the above command doesn't fail for me anymore.

Modified:
    clang-tools-extra/trunk/test/clang-rename/ClassTestReplacements.cpp

Modified: clang-tools-extra/trunk/test/clang-rename/ClassTestReplacements.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-rename/ClassTestReplacements.cpp?rev=273941&r1=273940&r2=273941&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/clang-rename/ClassTestReplacements.cpp (original)
+++ clang-tools-extra/trunk/test/clang-rename/ClassTestReplacements.cpp Mon Jun 27 16:04:53 2016
@@ -1,7 +1,8 @@
-// RUN: mkdir -p %T/fixes
+// RUN: rm -rf %t
+// RUN: mkdir -p %t/fixes
 // RUN: cat %s > %t.cpp
-// RUN: clang-rename -offset=225 -new-name=Hector -export-fixes=%T/fixes.yaml %t.cpp --
-// RUN: clang-apply-replacements %T
+// RUN: clang-rename -offset=256 -new-name=Hector -export-fixes=%t/fixes/clang-rename.yaml %t.cpp --
+// RUN: clang-apply-replacements %t
 // RUN: sed 's,//.*,,' %t.cpp | FileCheck %s
 class Cla  // CHECK: class Hector
 {




More information about the cfe-commits mailing list