brucem added a comment.
I downloaded a build of IWYU over the weekend, but haven't had the time to look
at using it yet!
I did find a couple of places where we can alter the included headers though
(patch coming this week) to include less and IWYU wouldn't catch those, so I'll
still do some
clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.
Very nice.
http://reviews.llvm.org/D12954
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
clayborg added a comment.
It would be great if there were an automated tool that could detect when any
#include wasn't needed for both headers and implementation files.
http://reviews.llvm.org/D12954
___
lldb-commits mailing list
This revision was automatically updated to reflect the committed changes.
Closed by commit rL248004: Reduce inclusion of clang headers. (authored by
brucem).
Changed prior to commit:
http://reviews.llvm.org/D12954?vs=35058=35097#toc
Repository:
rL LLVM
http://reviews.llvm.org/D12954
brucem created this revision.
brucem added a reviewer: clayborg.
brucem added a subscriber: lldb-commits.
With the recent changes to separate clang from the core structures
of LLDB, many inclusions of clang headers can be removed.
http://reviews.llvm.org/D12954
Files:
