This is an archive of the discontinued LLVM Phabricator instance.

[libc++][format] Updates feature-test macros.
ClosedPublic

Authored by Mordante on Sep 4 2022, 4:58 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rGb490c8a6623c: [libc++][format] Updates feature-test macros.
Summary

During the discussion on the SG-10 mailinglist regarding the format
feature-test macros voted in during the last plenary it turns out libc++
can't mark the format feature-test macro as implemented.

According to

https://isocpp.org/std/standing-documents/sd-6-sg10-feature-test-recommendations#__cpp_lib_format

the not yet implemented paper

P1361R2 Integration of chrono with text formatting

affects the feature test macro.

Note that P1361R2 doesn't mention the feature-test macro nor is there an
LWG-issue to address the issue. The reporter of the issue didn't recall
where this requirement exactly has been decided.

Diff Detail

Event Timeline

Mordante created this revision.Sep 4 2022, 4:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 4 2022, 4:58 AM
Herald added a subscriber: arichardson. · View Herald Transcript
Mordante requested review of this revision.Sep 4 2022, 4:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 4 2022, 4:58 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Sep 6 2022, 9:08 AM
ldionne added inline comments.
libcxx/utils/generate_feature_test_macro_components.py
318
This revision is now accepted and ready to land.Sep 6 2022, 9:08 AM
Mordante marked an inline comment as done.Sep 7 2022, 9:39 AM
This revision was automatically updated to reflect the committed changes.