This is an archive of the discontinued LLVM Phabricator instance.

[SystemZ][z/OS] Fix filesystem testing
ClosedPublic

Authored by muiez on Nov 5 2021, 10:40 AM.

Details

Reviewers
ldionne
Quuxplusone
Group Reviewers
Restricted Project
Commits
rG27ea67136ea0: [SystemZ][z/OS] Fix filesystem testing
Summary

This patch resolves many of the failures in the filesystems/ buckets in the libc++ tests. It adds the correct flag to fopen and marks a test case as unsupported. In particular, that test assumes time is stored as a 64 bit value when on MVS it is stored as 32 bit.

Diff Detail

Event Timeline

muiez requested review of this revision.Nov 5 2021, 10:40 AM
muiez created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptNov 5 2021, 10:40 AM
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald Transcript
ldionne accepted this revision.Nov 17 2021, 2:22 PM
ldionne added inline comments.
libcxx/test/support/filesystem_test_helper.h
198–200

While we're at it, let's format them correctly.

This revision is now accepted and ready to land.Nov 17 2021, 2:22 PM
muiez updated this revision to Diff 388200.Nov 18 2021, 7:41 AM

better format

ldionne accepted this revision.Nov 18 2021, 7:58 AM
This revision was automatically updated to reflect the committed changes.