[PATCH] D22208: [clang-tidy] Fixes to modernize-use-emplace
Piotr Padlewski via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 1 13:59:12 PDT 2016
Prazek added inline comments.
================
Comment at: clang-tidy/modernize/UseEmplaceCheck.h:36-37
@@ -32,1 +35,4 @@
+private:
+ std::vector<std::string> ContainersWithPushBack;
+ std::vector<std::string> SmartPointers;
};
----------------
aaron.ballman wrote:
> What about `llvm::make_range()`?
llvm::make_range do pretty much the same thing. The patch is in upstream, so feel free to try it, but either I am doing something wrong, or just SmallVector/ArrayRef have this limitations.
https://reviews.llvm.org/D22208
More information about the cfe-commits
mailing list