This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] Stop defining LIBCXX-WINDOWS-FIXME
ClosedPublic

Authored by mstorsjo on Apr 5 2022, 10:10 AM.

Details

Reviewers
ldionne
Mordante
Group Reviewers
Restricted Project
Commits
rG7594e236795f: [libcxx] [test] Stop defining LIBCXX-WINDOWS-FIXME
Summary

There are no such markings left - all of them have been fixed or
analyzed.

This closes llvm.org/PR32730 (github issue #32077).

Diff Detail

Event Timeline

mstorsjo created this revision.Apr 5 2022, 10:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2022, 10:10 AM
Herald added a subscriber: arichardson. · View Herald Transcript
mstorsjo requested review of this revision.Apr 5 2022, 10:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2022, 10:10 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
mstorsjo retitled this revision from [libcxx] [test] Remove LIBCXX-WINDOWS-FIXME to [libcxx] [test] Stop defining LIBCXX-WINDOWS-FIXME.Apr 6 2022, 5:44 AM
Mordante accepted this revision.Apr 6 2022, 9:05 AM

Nice that this project has been completed!
LGTM! The CI issue seems unrelated to this change, but either rerun the CI or keep an eye out after landing.

This revision is now accepted and ready to land.Apr 6 2022, 9:05 AM
This revision was landed with ongoing or failed builds.Apr 6 2022, 12:55 PM
This revision was automatically updated to reflect the committed changes.