Re: [Lldb-commits] [PATCH] D12954: Reduce inclusion of clang headers.

2015-09-21 Thread Bruce Mitchener via lldb-commits
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

Re: [Lldb-commits] [PATCH] D12954: Reduce inclusion of clang headers.

2015-09-18 Thread Greg Clayton via lldb-commits
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

Re: [Lldb-commits] [PATCH] D12954: Reduce inclusion of clang headers.

2015-09-18 Thread Greg Clayton via lldb-commits
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

Re: [Lldb-commits] [PATCH] D12954: Reduce inclusion of clang headers.

2015-09-18 Thread Bruce Mitchener via lldb-commits
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

[Lldb-commits] [PATCH] D12954: Reduce inclusion of clang headers.

2015-09-17 Thread Bruce Mitchener via lldb-commits
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: