This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] Do error printfs to stderr in filesystem tests
ClosedPublic

Authored by mstorsjo on Oct 19 2020, 2:52 AM.

Details

Summary

This makes them more readable in llvm-lit's output on failures.

This only applies the change on the filesystem test subdir.

Diff Detail

Event Timeline

mstorsjo created this revision.Oct 19 2020, 2:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 19 2020, 2:52 AM
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald Transcript
mstorsjo requested review of this revision.Oct 19 2020, 2:52 AM
ldionne accepted this revision.Oct 19 2020, 5:27 AM
ldionne added a subscriber: ldionne.

The reason why I originally used printf instead of fprintf(stderr) is that some platforms provide printf (via printing to some sort of uart port), but not a proper stderr. But I agree this change makes sense, and we can't make filesystem work on those platforms anyway.

This revision is now accepted and ready to land.Oct 19 2020, 5:27 AM