[PATCH] D141098: [clang-format][NFC] Set LineEnding to LF in config files

Tue Mar 14 19:30:40 PDT 2023

owenpan added a comment.

In D141098#4192785 <https://reviews.llvm.org/D141098#4192785>, @mehdi_amini wrote:

> In D141098#4111809 <https://reviews.llvm.org/D141098#4111809>, @owenpan wrote:
>
>> Can we just fix the buildbot so that it runs git-clang-format with `--binary`? Using clang-format-15 to format a patch to clang-format-16 source still looks wrong to me.
>
> The general principle is that developers of LLVM 16 should be able to use clang-format from the previous release (that is: one where we have prebuilt binaries available) to format patches in general.
> It does not seem reasonable to me that we should need to build clang-format ourselves for the purpose of writing patches to LLVM.

Agreed, except for the clang-format source, which its `.clang-format` files are part of.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141098