alexfh added a comment.

In https://reviews.llvm.org/D22656#493036, @Eugene.Zelenko wrote:

> Don't include DiagnosticIDs.h.


This header is just an example. I don't want anyone (including myself) manually 
figure out which of the headers inserted by IWYU actually need to be inserted. 
I'm also against adding #includes for every single transitively included 
header, since many of the headers can be considered "exported" by other 
headers. So, please, stop sending IWYU cleanups before reaching the general 
agreement and properly annotating LLVM and Clang headers.


Repository:
  rL LLVM

https://reviews.llvm.org/D22656



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to