[PATCH] D19865: [clang-tidy] - PerformanceUnnecesaryCopyInitialization - only trigger for decl stmts with single VarDecl.
Felix Berger via cfe-commits
cfe-commits at lists.llvm.org
Thu May 5 18:13:15 PDT 2016
flx added a comment.
In http://reviews.llvm.org/D19865#419905, @flx wrote:
> In http://reviews.llvm.org/D19865#419830, @alexfh wrote:
>
> > Is it a workaround to avoid breaking the code by incorrect fixes?
>
>
> Yes. We can't simply change the type of DeclStmt when we only look one of the VarDecls and how it is initialized.
Also, all tests still pass. Alex, do you have any particular concern with this approach?
Repository:
rL LLVM
http://reviews.llvm.org/D19865
More information about the cfe-commits
mailing list