[PATCH] D101378: [llvm, clang] Remove stdlib includes from .h files without `std::`

[Adrian McCarthy via Phabricator via llvm-commits]

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