[PATCH] D65912: [clang-tidy] Add new check for math constants

Mon Dec 30 06:20:06 PST 2019

JonasToth added a comment.
Herald added a subscriber: whisperity.

In D65912#1623375 <https://reviews.llvm.org/D65912#1623375>, @ZaMaZaN4iK wrote:

> > ... then I have a script that runs clang-tidy over all the compilation units in a compilation database
>
> Can you share this script please? :)

I did a bit of work on infrastructure for testing at some point, but never really finished something that works great and nice, just some script-ideas you can utilize.

You can use https://reviews.llvm.org/D54141 to deduplicate messages, especially such constants might be declared in headers that are included often.
Once you deduped the diagnostics you can pipe the output of `run-clang-tidy.py` in a file and do simple line counts and scroll though to get an idea of your checks output.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65912/new/

https://reviews.llvm.org/D65912