malcolm.parsons added a comment.
In https://reviews.llvm.org/D27166#622103, @Prazek wrote:
> There is still one more problem:
>
> /home/prazek/llvm/lib/Analysis/ScalarEvolution.cpp:2442:11: warning: use
> auto when initializing with a template cast to avoid duplicating the type
> name [modernize-use-auto]
> const auto **O = SCEVAllocator.Allocate<const SCEV *>(Ops.size());
> ^
> auto
Any suggestions for rewriting this matcher?
// Skip declarations that are already using auto.
unless(has(varDecl(anyOf(hasType(autoType()),
hasType(pointerType(pointee(autoType()))),
hasType(referenceType(pointee(autoType())))))))
> There is also problem with function pointers
>
> /home/prazek/llvm/lib/ExecutionEngine/MCJIT/MCJIT.cpp:520:9: warning: use
> auto when initializing with a cast to avoid duplicating the type name
> [modernize-use-auto]
> int (*PF)(int, char **, const char **) =
> ^
> auto
>
>
The warning is correct, but the fixit is wrong.
Suppress fixit for function pointers?
https://reviews.llvm.org/D27166
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
