This is an archive of the discontinued LLVM Phabricator instance.

[VE] Remove output to /dev/stdout
ClosedPublic

Authored by Jake-Egan on Mar 16 2022, 6:39 AM.

Details

Summary

Sending output to /dev/stdout on AIX gets an llc permission denied error, so this patch removes this from the tests.

Diff Detail

Event Timeline

Jake-Egan created this revision.Mar 16 2022, 6:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 16 2022, 6:39 AM
Jake-Egan requested review of this revision.Mar 16 2022, 6:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 16 2022, 6:39 AM
Jake-Egan edited the summary of this revision. (Show Details)Mar 16 2022, 6:42 AM
Jake-Egan added reviewers: simoll, kaz7.
simoll accepted this revision.Mar 16 2022, 6:59 AM

I always wondered why we are doing the <..

This revision is now accepted and ready to land.Mar 16 2022, 6:59 AM

LGTM by the way. The use of /dev/stdout would cause grief for Windows too?

This revision was landed with ongoing or failed builds.Mar 16 2022, 8:42 AM
This revision was automatically updated to reflect the committed changes.