<div dir="ltr">When I brought this up, I think the recommendation was to put these into separate include groups (i.e. add a newline between atlbase.h and windows.h -- and the DIA headers are already in a separate block). That seems like a good approach to me.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 11, 2016 at 4:20 AM, Chandler Carruth via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yes, anything like this would be a bug in the logic and I'll file it rather than submit it.</div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote"><div dir="ltr">On Fri, Mar 11, 2016 at 10:18 AM Ismail Donmez via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Mar 11, 2016 at 11:14 AM, Chandler Carruth via cfe-dev<br>
<<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br>
> The crazy clang-format folks have taught it to sort includes, and clang-tidy<br>
> has a nice way to run it across all of LLVM and Clang.<br>
><br>
> Before I just submit the fixes (and any bug reports to the clang-format<br>
> folks if it sorts things weirdly) I wanted to double check that folks would<br>
> be ok with this. My understanding is that the sorting should match the LLVM<br>
> conventions, but if not, I'll make sure to get that fixed first.<br>
<br>
Should take care of windows headers, part of DIASupport.h:<br>
<br>
// atlbase.h has to come before windows.h<br>
#include <atlbase.h><br>
#include <windows.h><br>
<br>
// DIA headers must come after windows headers.<br>
                                                   #include<br>
<cvconst.h><br>
#include <dia2.h><br>
<br>
Sorting those would break stuff.<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div>
</div></div><br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>