This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Mark namespace std with default visibility and remove template and enum visibility macros
AbandonedPublic

Authored by philnik on Jun 11 2023, 8:20 PM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Summary

This gives everything inside namespace std default visibility and removes macros which give types default visibility. This isn't a problem, since all the functions should be marked _LIBCPP_HIDE_FROM_ABI anyways, and everything else should have default visibilty.

Diff Detail

Event Timeline

philnik created this revision.Jun 11 2023, 8:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 11 2023, 8:20 PM
Herald added a subscriber: arphaman. · View Herald Transcript
philnik requested review of this revision.Jun 11 2023, 8:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 11 2023, 8:20 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik updated this revision to Diff 530517.Jun 12 2023, 8:29 AM

Update ignore_format.txt

philnik updated this revision to Diff 530520.Jun 12 2023, 8:37 AM

Remove more files from ignore_format.txt

philnik abandoned this revision.Jun 12 2023, 11:51 AM