[PATCH] D68839: [lit] Fix internal diff's --strip-trailing-cr and use it

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 14 18:50:40 PDT 2019


jdenny updated this revision to Diff 224947.
jdenny added a comment.

- Added `--strip-trailing-cr` to another test that needs it.
- Removed some accidental FileCheck options.
- Rebased onto master so it doesn't depend on D66574 <https://reviews.llvm.org/D66574> anymore.  Thus, it modifies TestRunner.py instead of diff.py.

Another review shouldn't be necessary. I plan to push tomorrow.


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

https://reviews.llvm.org/D68839

Files:
  llvm/test/MC/ARM/preserve-comments-arm.s
  llvm/test/MC/AsmParser/preserve-comments.s
  llvm/test/tools/llvm-cxxmap/remap.test
  llvm/test/tools/llvm-profdata/profile-symbol-list.test
  llvm/test/tools/llvm-profdata/roundtrip.test
  llvm/test/tools/llvm-profdata/sample-remap.test
  llvm/utils/lit/lit/TestRunner.py
  llvm/utils/lit/tests/Inputs/shtest-shell/diff-in.dos
  llvm/utils/lit/tests/Inputs/shtest-shell/diff-in.unix
  llvm/utils/lit/tests/Inputs/shtest-shell/diff-strip-trailing-cr.txt
  llvm/utils/lit/tests/max-failures.py
  llvm/utils/lit/tests/shtest-shell.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68839.224947.patch
Type: text/x-patch
Size: 8892 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191015/02fcce92/attachment.bin>


More information about the llvm-commits mailing list