VitaNuo accepted this revision. VitaNuo added a comment. This revision is now accepted and ready to land.
Thanks! ================ Comment at: clang-tools-extra/clang-tidy/misc/IncludeCleanerCheck.cpp:49 SourceLocation SymRefLocation; + include_cleaner::Symbol Sym; include_cleaner::Header Missing; ---------------- Consider making the whole `include_cleaner::SymbolReference` a field instead, since you're already using two out of its three fields here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D153013/new/ https://reviews.llvm.org/D153013 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
