aaronpuchert added a comment.

In D124563#3479819 <https://reviews.llvm.org/D124563#3479819>, @MForster wrote:

> See D124606 <https://reviews.llvm.org/D124606>. `-text` seems to be the right 
> compromise.

To quote an earlier comment from this thread:

In D124563#3478625 <https://reviews.llvm.org/D124563#3478625>, @smeenai wrote:

> I *think* this would mean that if you're on Windows and have `core.autocrlf` 
> set to `input`, when you commit changes to this files, Git will convert them 
> back to LF line endings. Not 100% sure of that though.

Which convinced me that D97625 <https://reviews.llvm.org/D97625> was right: we 
do want the line endings to remain CRLF, and not have them accidentally 
destroyed. With `-text` we say that we don't care, which is not true.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124563

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to