This is an archive of the discontinued LLVM Phabricator instance.

Point replay file to non-existent dummy
ClosedPublic

Authored by stevewan on Oct 28 2021, 12:10 PM.

Details

Summary

Operating systems such as AIX allow open and read on directories, passing in a direcotry as the replay file triggers Invalid remark format instead of Could not open remarks file: Is a directory. This patch substitutes the directory with a non-existent filename. The current filecheck should still work as-is.

Diff Detail

Event Timeline

stevewan created this revision.Oct 28 2021, 12:10 PM
stevewan requested review of this revision.Oct 28 2021, 12:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 28 2021, 12:10 PM
stevewan edited the summary of this revision. (Show Details)Oct 28 2021, 12:13 PM
stevewan added reviewers: modimo, yousufhali, Whitney.
modimo accepted this revision.Oct 28 2021, 12:17 PM

Thanks, I'll make sure to avoid pointing at directories to validate these things in the future.

This revision is now accepted and ready to land.Oct 28 2021, 12:17 PM

Thanks, I'll make sure to avoid pointing at directories to validate these things in the future.

Thank you. I'll see if other reviewers have any comments too.

Whitney accepted this revision.Oct 28 2021, 1:08 PM
This revision was automatically updated to reflect the committed changes.