[PATCH] D23434: Don't allow llvm-include-order to intermingle includes from different files.

Zachary Turner via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 12 04:33:59 PDT 2016


Neither does clang-tidy's right?

Incidentally that's exactly what i was trying to add support for. Lldb has
a mass reformat coming up, and as part of that we are defining an include
ordering. But at the moment it won't do this, so we will be left fixing
this by hand or not fixing it at all


On Fri, Aug 12, 2016 at 12:36 AM Daniel Jasper <djasper at google.com> wrote:

> djasper added a comment.
>
> I think we should entirely drop this implementation of the check and let
> it just check #includes with clang-format. clang-format's implementation
> isn't a strict superset, e.g. won't sort between multiple blocks, but
> that's intentional for now.
>
>
> https://reviews.llvm.org/D23434
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160812/5f8a00ec/attachment.html>


More information about the cfe-commits mailing list