[PATCH] D30210: [include-fixer] Add usage count to find-all-symbols.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 24 02:54:40 PST 2017


hokein added a comment.

Thanks, still LGTM with one nit.



================
Comment at: include-fixer/find-all-symbols/FindAllSymbols.cpp:262
+  if (Filename != "") {
+    Reporter->reportSymbols(Filename, std::move(FileSymbols));
+    FileSymbols = {};
----------------
We don't need `std::move` right now.


https://reviews.llvm.org/D30210





More information about the cfe-commits mailing list