[cfe-dev] [llvm-dev] Any objections to re-sorting #include lines with clang-format/clang-tidy?
Chandler Carruth via cfe-dev
cfe-dev at lists.llvm.org
Fri Mar 11 17:05:49 PST 2016
On Sat, Mar 12, 2016 at 2:02 AM Chris Lattner via llvm-dev <
llvm-dev at lists.llvm.org> wrote:
> > On Mar 11, 2016, at 1:14 AM, Chandler Carruth via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
> > The crazy clang-format folks have taught it to sort includes, and
> clang-tidy has a nice way to run it across all of LLVM and Clang.
> > Before I just submit the fixes (and any bug reports to the clang-format
> folks if it sorts things weirdly) I wanted to double check that folks would
> be ok with this. My understanding is that the sorting should match the LLVM
> conventions, but if not, I'll make sure to get that fixed first.
> Does it do it the “right” way? I think it is important for a .cpp file to
> include its corresponding header first (to ensure it stays self contained).
I don't know for sure, but I'm completely on the same page. I'm not going
to radically change the sorting rules at all. Any differences there will
just be bugs against the tool that I'll file.
So I'm not going to submit a non-"right" sort. =] I'll just file bugs there.
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-dev