kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land.
thanks, lgtm! ================ Comment at: clang-tools-extra/include-cleaner/include/clang-include-cleaner/Analysis.h:47 /// (e.g. IWYU pragmas). -void walkUsed(llvm::ArrayRef<Decl *> ASTRoots, UsedSymbolCB CB); +void walkUsed(const SourceManager &, llvm::ArrayRef<Decl *> ASTRoots, + llvm::ArrayRef<SymbolReference> MacroRefs, UsedSymbolCB CB); ---------------- nit: you mind moving sourcemanager to be after roots and macrorefs? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137644/new/ https://reviews.llvm.org/D137644 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
