[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