[PATCH] D83711: [AIX] Remove diff -a option on llvm-cov.test

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 13 15:30:19 PDT 2020


hubert.reinterpretcast accepted this revision.
hubert.reinterpretcast marked 2 inline comments as done.
hubert.reinterpretcast added a comment.
This revision is now accepted and ready to land.

Yes, none of the `.gcov` reference files appear to be anything other than plain ASCII text. Change LGTM.



================
Comment at: llvm/test/tools/llvm-cov/llvm-cov.test:16
 RUN: FileCheck %s --check-prefix=C --match-full-lines --strict-whitespace < test.cpp.gcov
-RUN: diff -aub test_no_options.h.gcov test.h.gcov
+RUN: diff -ub test_no_options.h.gcov test.h.gcov
 RUN: count 0 < %t.err
----------------
Confirmed:
```
test_no_options.h.gcov: ascii text
```


================
Comment at: llvm/test/tools/llvm-cov/llvm-cov.test:160
+RUN: diff -ub test_no_gcda.cpp.gcov test.cpp.gcov
+RUN: diff -ub test_no_gcda.h.gcov test.h.gcov
 NO-GCDA:       File 'test.cpp'
----------------
Confirmed:
```
test_no_gcda.cpp.gcov: ascii text
test_no_gcda.h.gcov: ascii text
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83711/new/

https://reviews.llvm.org/D83711





More information about the llvm-commits mailing list