cjdb added a comment.

In D106394#2893681 <https://reviews.llvm.org/D106394#2893681>, @Quuxplusone 
wrote:

> If IWYU has already developed a de facto standard for this information, I 
> would recommend libc++ just use it (perhaps with no changes needed in clang 
> at all). This would have the benefit of getting libc++ to work with IWYU 
> out-of-the-box, instead of having to go beg IWYU to implement support for a 
> new `#pragma include_instead`.  Is any IWYU developer aware of D106394 
> <https://reviews.llvm.org/D106394> yet? You should definitely try to get them 
> involved here.

This suggestion misses the fact that this change issues a warning to users who 
include detail system headers and the fact that Clang can use this to improve 
its diagnostics.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106394/new/

https://reviews.llvm.org/D106394

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

Reply via email to