amccarth added a comment. A drive-by look.
================ Comment at: clang/include/clang/Tooling/Transformer/Parsing.h:24 #include <functional> -#include <string> ---------------- `<functional>` appears unnecessary as well. And while this doesn't require `<string>` it does require `llvm/ADT/StringRef.h`. ================ Comment at: llvm/include/llvm/DebugInfo/CodeView/GlobalTypeTableBuilder.h:72 template <typename CreateFunc> TypeIndex insertRecordAs(GloballyHashedType Hash, size_t RecordSize, CreateFunc Create) { ---------------- `size_t` may have been coming indirectly from `<memory>`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101378/new/ https://reviews.llvm.org/D101378 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits