[PATCH] D27166: [clang-tidy] Enhance modernize-use-auto to templated function casts

[Malcolm Parsons via Phabricator via cfe-commits]

malcolm.parsons added a comment.

In https://reviews.llvm.org/D27166#622072, @Prazek wrote:

> Have you run this check on LLVM & Clang to check if it works?


I ran it on some of clang-tools-extra:

  unittests/clang-tidy/ClangTidyDiagnosticConsumerTest.cpp:17
  -    const VarDecl *Var = Result.Nodes.getNodeAs<VarDecl>("var");
  +    const auto *Var = Result.Nodes.getNodeAs<VarDecl>("var");



>   lib/Analysis/AssumptionCache.cpp:120
>   -    for (const BasicBlock &B : cast<Function>(*I.first))
>   +    for (const BasicBlock &B : auto<Function>(*I.first))

The check should be ignoring implicit VarDecls.


https://reviews.llvm.org/D27166



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits