[clang-tools-extra] [clang-tidy] add misc-math-constant check (PR #66583)

via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 16 11:27:36 PDT 2023


5chmidti wrote:

- How do I correctly parse the values in a macro? While the test locally works, it doesn't in CI. I also don't differentiate between what kind of numeric constant the token is, which might be another problem. Any pointers on how to do this correctly are greatly appreciated.
- Macros currently don't distinguish between `float`, `double` and `long double` like the AST-based check does. The values matched in the PPCallback should be able to provide the `..._v<>` versions of replacements as well. The question about how to detect the difference is probably related to the point above.

https://github.com/llvm/llvm-project/pull/66583


More information about the cfe-commits mailing list