This is an archive of the discontinued LLVM Phabricator instance.

[libc++][format] Mark range formaters as complete.
ClosedPublic

Authored by Mordante on Apr 9 2023, 4:29 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rGb23735407008: [libc++][format] Mark range formaters as complete.
Summary

The __cpp_lib_format_ranges feature-test macro only depends on P2286R8
and P2585R0. Note since LWG3750 only affects these two C++23 papers
there is nothing to do for older language versions.

(The __cpp_lib_format feature-test macro depends on the incomplete
formatting for chrono. So this part can't be marked as complete yet.)

This completes

  • P2286R8 Formatting ranges
  • P2585R0 Improving default container formatting

This partly implements

  • LWG3750 Too many papers bump __cpp_lib_format

Diff Detail

Event Timeline

Mordante created this revision.Apr 9 2023, 4:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2023, 4:29 AM
Herald added a subscriber: arichardson. · View Herald Transcript
Mordante requested review of this revision.Apr 9 2023, 4:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2023, 4:29 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Apr 11 2023, 8:57 AM
ldionne added a subscriber: ldionne.

LGTM w/ changes!

libcxx/utils/generate_feature_test_macro_components.py
334–335

If you rebase onto main, I think what you want here is just !defined(_LIBCPP_HAS_NO_INCOMPLETE_FORMAT).

340–341

These ones should also be updated to just !defined(_LIBCPP_HAS_NO_INCOMPLETE_FORMAT). I think you created the patch that added this before I changed __availability.

This revision is now accepted and ready to land.Apr 11 2023, 8:57 AM
Mordante marked 2 inline comments as done.Apr 11 2023, 9:46 AM
Mordante added inline comments.
libcxx/utils/generate_feature_test_macro_components.py
340–341

Yes and since there were no merge conflicts there was no notification of the issue. But at least good we spotted it during review!

This revision was landed with ongoing or failed builds.Apr 11 2023, 9:47 AM
This revision was automatically updated to reflect the committed changes.
Mordante marked an inline comment as done.