This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] Move use of statvfs to helper header
ClosedPublic

Authored by mstorsjo on Oct 21 2020, 3:46 AM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rG78ba1e93a6f2: [libcxx] [test] Move use of statvfs to helper header
Summary

Implement the corresponding thing using windows functions as well.

Diff Detail

Event Timeline

mstorsjo created this revision.Oct 21 2020, 3:46 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 21 2020, 3:46 AM
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald Transcript
mstorsjo requested review of this revision.Oct 21 2020, 3:46 AM
ldionne requested changes to this revision.Oct 21 2020, 5:20 AM
ldionne added a subscriber: ldionne.
ldionne added inline comments.
libcxx/test/support/filesystem_test_helper.h
87

We should use assert here.

This revision now requires changes to proceed.Oct 21 2020, 5:20 AM
mstorsjo updated this revision to Diff 299646.Oct 21 2020, 5:31 AM

Using assert() instead of TEST_CHECK() in the helper header.

ldionne accepted this revision.Oct 21 2020, 5:57 AM
This revision is now accepted and ready to land.Oct 21 2020, 5:57 AM
This revision was automatically updated to reflect the committed changes.