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

Reply via email to