[PATCH] D141495: [clangd] Suppress clang-tidy warnings for code spelled in system macros

Nathan Ridge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 11 07:49:07 PST 2023


nridge added inline comments.


================
Comment at: clang-tools-extra/clangd/ParsedAST.cpp:545
           }
+          // Match behavior for clang-tidy --system-headers=0 (the default).
+          if (Info.hasSourceManager() &&
----------------
Why not check `CTContext->getOptions().SystemHeaders`, since we have ready access to it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141495



More information about the cfe-commits mailing list