[PATCH] D52360: [clang-tidy] Fix for performance-unnecessary-value-param, performance-unnecessary-copy-initialization and performance-for-range-copy

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 26 05:17:37 PDT 2018


lebedev.ri added a comment.

In https://reviews.llvm.org/D52360#1246440, @baloghadamsoftware wrote:

> Config option is a good idea but it must not be empty by default. The checker must ignore all pointer and references by default based on their names.


I disagree, it **must** not have false-negatives by default.
It **may** have false-positives, that are controllable via the options.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52360





More information about the cfe-commits mailing list