[PATCH] D31532: Replace wc -l with count

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 31 09:45:28 PDT 2017


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm This just looks like someone knew more about shell than LLVM testing utilities. :)



================
Comment at: compiler-rt/test/asan/TestCases/Linux/coverage-missing.cc:11
 // RUN: rm *.sancov
-// RUN: [ $(cat main.txt | wc -l) == 1 ]
+// RUN: cat main.txt | count 1
 // RUN: %env_asan_opts=coverage=1:coverage_dir=%T/coverage-missing %run %t x
----------------
Maybe make this `count 1 < main.txt` and save a process?


https://reviews.llvm.org/D31532





More information about the llvm-commits mailing list