This is an archive of the discontinued LLVM Phabricator instance.

Replace wc -l with count
ClosedPublic

Authored by rafael on Mar 31 2017, 6:16 AM.

Details

Reviewers
rnk
Summary

This is a far more common way in llvm of counting lines in tests

Diff Detail

Event Timeline

rafael created this revision.Mar 31 2017, 6:16 AM
rnk accepted this revision.Mar 31 2017, 9:45 AM

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

compiler-rt/test/asan/TestCases/Linux/coverage-missing.cc
11

Maybe make this count 1 < main.txt and save a process?

This revision is now accepted and ready to land.Mar 31 2017, 9:45 AM
rafael closed this revision.Mar 31 2017, 10:03 AM