[libc-commits] [libc] [libc] start fix readability-identifier-naming lints (PR #83342)
via libc-commits
libc-commits at lists.llvm.org
Wed Feb 28 13:54:10 PST 2024
lntue wrote:
> Otherwise, if we don't update the clang-tidy rules (and instead have constexpr functions be lowercase, the current number of unique lints here is 11.
>
> So by that measure, I think that it makes sense to:
> 1. codify that we use lower_case for constexpr functions.
> 2. fix the 11 violations
>
> Rather then codify it as upper case and fix the 170 resulting violations.
How about option 3: mark these functions `consteval` and add C++20 standard when needed? Or add TODO's for option 3?
https://github.com/llvm/llvm-project/pull/83342
More information about the libc-commits
mailing list