[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