[PATCH] D33470: [clang-tidy] Add misc-default-numerics
Piotr Padlewski via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Jun 24 03:12:57 PDT 2017
Prazek added a comment.
In https://reviews.llvm.org/D33470#764846, @aaron.ballman wrote:
> Once you fix the typo in the check, can you run it over some large C++ code bases to see if it finds any results?
I tried it on LLVM code base (after fixing bug with the numeric_limits name) and it didn't find anything suspisious.
Unfortunatelly I don't have enough time to try it on different codebases, but I am weiling to fix any bug with this check if it would happen in the future.
The release 5.0 is near, so I would like to push it upstream. Does it sound good to you?
https://reviews.llvm.org/D33470
More information about the cfe-commits
mailing list