We normally use the __libcpp_ prefix to backport functions, but that doesn't really make sense here. The functions are always available. Using __constexpr_ makes the intended use of the functions clear.
Details
Details
- Reviewers
ldionne Mordante var-const - Group Reviewers
Restricted Project - Commits
- rGdd28e22953ad: [libc++][NFC] Rename __libcpp_*_or_builtin to __constexpr_*
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Event Timeline
Comment Actions
Typically our internal functions like these are name __libcpp_foo, not __libcpp_foo_or_builtin. I see you now rename them to __constexpr_foo. Can you add comment to the commit message why you decided to use this new naming scheme?
I see the CI is red, but it just seems an internal test failure.