[PATCH] D19482: [include-fixer] Add a find-all-symbols tool for include-fixer.
Daniel Jasper via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 27 07:26:35 PDT 2016
djasper accepted this revision.
djasper added a comment.
Looks good.
================
Comment at: include-fixer/find-all-symbols/FindAllSymbols.cpp:39
@@ +38,3 @@
+ } else {
+ const auto *RD = llvm::cast<RecordDecl>(Context);
+ Symbol->Contexts.emplace_back(SymbolInfo::Record, RD->getName().str());
----------------
I'd still remove a "llvm::"..
http://reviews.llvm.org/D19482
More information about the cfe-commits
mailing list