<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 6, 2017 at 3:56 AM Chandler Carruth via Phabricator via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@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">chandlerc created this revision.<br>
Herald added subscribers: mcrosier, sanjoy, klimek.<br>
<br>
This really is a collection of improvements to the rules for LLVM<br>
include sorting:<br>
<br>
- We have gmock headers now, so it adds support for those to one of the categories.<br>
- LLVM does use 'FooTest.cpp' files to test 'Foo.h' so it adds that suffix for finding a main header.<br>
- At times the test file's case may not match the header file's case, so adds support for case-insensitive regex matching of these things.<br></blockquote><div><br>Curious - where does this last one come up? Can we fix those, or do they look good/intentional/worth keeping?<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
With this set of changes, I can't spot any misbehaviors when re-sorting<br>
all of LLVM's unittest '#include' lines.<br>
<br>
Note that I don't know the first thing about testing clang-format, so please<br>
feel free to tell me what all I need to be doing here.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D33932" rel="noreferrer" target="_blank">https://reviews.llvm.org/D33932</a><br>
<br>
Files:<br>
  include/clang/Format/Format.h<br>
  lib/Format/Format.cpp<br>
  unittests/Format/SortIncludesTest.cpp<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>