hokein added a comment. Mostly good. A few nits.
================ Comment at: clang-tidy/performance/UnnecessaryValueParamCheck.cpp:103 // Do not trigger on non-const value parameters when they are not only used as // const. ---------------- This comment needs to be updated. ================ Comment at: clang-tidy/performance/UnnecessaryValueParamCheck.cpp:108 return; + if (const auto *Ctor = dyn_cast<CXXConstructorDecl>(Function)) { + for (const auto *Init : Ctor->inits()) { ---------------- Is this a new fix or a special case not being caught by `ExprMutationAnalyzer`? Do we have a test case covered it? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D50102 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits