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

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 12 08:20:52 PDT 2016


alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

The patch LG.

In https://reviews.llvm.org/D23434#513533, @djasper wrote:

> 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.


Since sorting across blocks is frequently needed to make include order consistent with the LLVM coding standards, it makes sense to keep this check until clang-format is able to sort across blocks. Especially, since Zachary is specifically interested in this functionality.


https://reviews.llvm.org/D23434





More information about the cfe-commits mailing list