[PATCH] D117897: [split-file] Respect input file's line endings

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 21 10:07:43 PST 2022


MaskRay added a comment.

I think llvm-project tests normally should all use '\n', but this change may make the utility more useful outside llvm-project, so looks good.



================
Comment at: llvm/test/tools/split-file/basic.crlf.test:1
+#--- aa
+aa
----------------
We do not these many tests. One crlf file is sufficient.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117897



More information about the llvm-commits mailing list