[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