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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits