This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Adds __cpp_lib_constexpr_algorithms to utility.
ClosedPublic

Authored by Mordante on Dec 20 2022, 9:03 AM.

Details

Summary

Implements:

  • LWG3792 __cpp_lib_constexpr_algorithms should also be defined in <utility>

Depends on D140407

Diff Detail

Event Timeline

Mordante created this revision.Dec 20 2022, 9:03 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 20 2022, 9:03 AM
Herald added a subscriber: arichardson. · View Herald Transcript
Mordante requested review of this revision.Dec 20 2022, 9:03 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 20 2022, 9:03 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik accepted this revision.Dec 20 2022, 9:09 AM
philnik added a subscriber: philnik.

LGTM with green CI.

libcxx/docs/Status/Cxx2bIssues.csv
234

I think we could claim this as implemented earlier, but I'm OK with claiming it in 16 too.

This revision is now accepted and ready to land.Dec 20 2022, 9:09 AM
Mordante marked an inline comment as done.Dec 20 2022, 9:27 AM

Thanks for the review!

libcxx/docs/Status/Cxx2bIssues.csv
234

This issue is specifically about the missing feature-test macro. I agree the feature itself has been implemented earlier.

philnik added inline comments.Dec 20 2022, 9:30 AM
libcxx/docs/Status/Cxx2bIssues.csv
234

I mean that, since we provide all FTMs everywhere, we already provided this one in <utility> before.

Mordante marked an inline comment as done.Dec 20 2022, 11:00 AM
Mordante added inline comments.
libcxx/docs/Status/Cxx2bIssues.csv
234

Ah yes I see. From that point of view we indeed added support before.

ldionne accepted this revision.Dec 21 2022, 6:19 AM
This revision was automatically updated to reflect the committed changes.