Re: [PATCH] D19482: [include-fixer] Add a find-all-symbols tool for include-fixer.

[Daniel Jasper via cfe-commits]

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
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits