aaron.ballman added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/llvmlibc/RestrictSystemLibcHeadersCheck.cpp:40 + SrcMgr::CharacteristicKind FileType) { + if (SrcMgr::isSystem(FileType)) { + if (!SM.isInMainFile(HashLoc)) { ---------------- njames93 wrote: > abrachet wrote: > > Could you whitelist the freestanding/compiler provided headers like stddef, > > stdatomic... > Or have a user configurable whitelist It should be configurable and named something other than whitelist. I'd recommend `AllowedHeaders` or something along those lines. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75332/new/ https://reviews.llvm.org/D75332 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits