[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