There are constexpr versions for the different __invoke functions, which seem to be identical other than begin constexpr since C++11 instead of being constexpr since C++20.
Details
Details
- Reviewers
ldionne Mordante EricWF var-const - Group Reviewers
Restricted Project - Commits
- rG1b9c5f60aabd: [libc++] Remove redundant __invoke_constexpr functions
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Looks like a nice cleanup to me, thanks!
libcxx/test/libcxx/utilities/function.objects/func.require/bullet_1_2_3.pass.cpp | ||
---|---|---|
274 | Can we test this function as constexpr or is there other test coverage? |
Comment Actions
This looks correct to me if CI is green, but I would like to see Mark's suggestion addressed.
Can we test this function as constexpr or is there other test coverage?