[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