[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