[PATCH] D52727: [clang-tidy] White List Option for performance-unnecessary-value-param, performance-unnecessary-copy-initialization and performance-for-range-copy

[Jonas Toth via Phabricator via cfe-commits]

JonasToth added a comment.

LG in principle, just the SmallVec thing could be done if you agree. I don't 
insist on it, but it looks like a performance benefit to me.



================
Comment at: clang-tidy/performance/UnnecessaryCopyInitialization.h:41
                               ASTContext &Context);
+  const std::vector<std::string> AllowedTypes;
 };
----------------
I think these lists for the allowed types could even be SmallVectors, as I 
wouldn't expect so many entries.


https://reviews.llvm.org/D52727



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits