<p dir="ltr">The check's implementation will be replaced by a simple call to clang tidy. It will remain a check in clang tidy to continue to cater to both use cases.</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Aug 12, 2016 6:19 PM, "Zachary Turner" <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">That's actually the reason I think it makes more sense in clang tidy.  It can be a configuration option, off by default, and since there is more control over whether to apply fixits, and it doesn't apply fixits by default, it would be easier to iterate on the experimental nature of it without messing up code <br><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 12, 2016 at 9:14 AM Alexander Kornienko <<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">alexfh added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D23434#513839" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D23434#513839</a>, @djasper wrote:<br>
<br>
> I think we got confused. We once had tried to write an experimental separate check to comply with Google's style guide. If you want to fiddle around with that, contact me, I can send you pointers. But as I mentioned we moved away from that. And I think it makes more sense to re-create the sort-across-blocks functionality in clang-format and not in clang-tidy.<br>
<br>
<br>
Yep, we definitely got confused. That experimental check actually implemented cross-block sorting, but this caused a bunch of issues. Which makes me think that proper implementation of cross-block include sorting is challenging be it in clang-format or clang-tidy. Clang-format probably makes it even more complex, since a higher safety of transformations is expected from it.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23434" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D23434</a><br>
<br>
<br>
<br>
</blockquote></div>
</blockquote></div></div>