[cfe-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 01:20:24 PST 2016


Yes, anything like this would be a bug in the logic and I'll file it rather
than submit it.

On Fri, Mar 11, 2016 at 10:18 AM Ismail Donmez via cfe-dev <
cfe-dev at lists.llvm.org> wrote:

> On Fri, Mar 11, 2016 at 11:14 AM, Chandler Carruth via cfe-dev
> <cfe-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.
>
> Should take care of windows headers, part of DIASupport.h:
>
> // atlbase.h has to come before windows.h
> #include <atlbase.h>
> #include <windows.h>
>
> // DIA headers must come after windows headers.
>                                                    #include
> <cvconst.h>
> #include <dia2.h>
>
> Sorting those would break stuff.
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20160311/5708a28f/attachment.html>


More information about the cfe-dev mailing list