This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Unconditionally provide the open-with-wchar_t extension
AbandonedPublic

Authored by ldionne on Aug 23 2021, 12:55 PM.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

The extension is enabled and apparently quite useful on Windows. While
I think extensions are generally evil, conditional extensions are even
worse, so I'd rather enable this extension on all platforms.

Diff Detail

Event Timeline

ldionne requested review of this revision.Aug 23 2021, 12:55 PM
ldionne created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 23 2021, 12:55 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne updated this revision to Diff 368449.Aug 24 2021, 1:27 PM
ldionne retitled this revision from [libc++] Remove mentions of _LIBCPP_HAS_OPEN_WITH_WCHAR in the test suite to [libc++] Unconditionally provide the open-with-wchar_t extension.
ldionne edited the summary of this revision. (Show Details)

Unconditionally enable the extension as discussed in https://reviews.llvm.org/D42225

ldionne abandoned this revision.Oct 5 2021, 11:55 AM

Turns out this was a bit naive, _wfopen isn't provided outside of Windows so we can't implement this.