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

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

Reply via email to