[PATCH] D30210: [include-fixer] Add usage count to find-all-symbols.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 23 11:48:11 PST 2017
hokein added inline comments.
================
Comment at: unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp:40
+ void reportSymbols(llvm::StringRef FileName,
+ SymbolInfo::SignalMap NewSymbols) override {
+ for (const auto &Entry : NewSymbols)
----------------
A new catch: `NewSymbols` should be passed by reference, otherwise a copy will be generated.
https://reviews.llvm.org/D30210
More information about the cfe-commits
mailing list