DmitryPolukhin added a comment.

In D90835#2808153 <https://reviews.llvm.org/D90835#2808153>, @lebedev.ri wrote:

> This needs to explain why the existing functionality isn't sufficient - if 
> the header is really not from this project,
> then it should be included via `-isystem`, and that will naturally suppress 
> all diagnostics from it.

The diagnostics are reported in the place of macro expansion i.e. user sources. 
It doesn't matter how they get included via -isystem or -I.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90835

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D90835: [clang-tid... Roman Lebedev via Phabricator via cfe-commits
    • [PATCH] D90835: [clan... Dmitry Polukhin via Phabricator via cfe-commits

Reply via email to