[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