malcolm.parsons added a comment. In https://reviews.llvm.org/D26453#596254, @aaron.ballman wrote:
> (This part of the check is one half modernization, one half performance, and > one half correctness, depending on which lens you view the code through.) The performance part should be handled by performance-unnecessary-value-param. It should warn in this situation, but not create a fix that would conflict with modernize-pass-by-value's fix. https://reviews.llvm.org/D26453 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
