This is an archive of the discontinued LLVM Phabricator instance.

[libc++][format] Removes format sources.
ClosedPublic

Authored by Mordante on May 7 2023, 11:58 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rGdc95245e69a1: [libc++][format] Removes format sources.
Summary

The source file is used to anchor the destructor of format_error. When
format is moved from experimental to stable this code would move to the
dylib. One issue with code in the dylib is that it can't be used in
constexpr context. There is a proposal to make format work during
constant evaluation

P2758 Emitting messages at compile time

This paper has initially been received favourable by EWG. Therefore move
the code to the header. This also avoids possible availability issues on
Mac back deployment targets.

Note it is expected that format will no longer be experimental with the
next LLVM release.

Diff Detail

Event Timeline

Mordante created this revision.May 7 2023, 11:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 7 2023, 11:58 AM
Mordante updated this revision to Diff 521886.May 13 2023, 2:23 AM

Rebased and CI fixes.

Mordante updated this revision to Diff 521896.May 13 2023, 3:36 AM

Fixes CI.

Mordante published this revision for review.May 13 2023, 8:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2023, 8:22 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.May 16 2023, 9:07 AM

Nice cleanup, thanks!

This revision is now accepted and ready to land.May 16 2023, 9:07 AM
This revision was automatically updated to reflect the committed changes.
libcxx/src/CMakeLists.txt