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