Page MenuHomePhabricator

[lld-macho][nfc] Fixed test so it output to %t/ rather than current directory.
ClosedPublic

Authored by oontvoo on Mar 24 2021, 8:50 AM.

Details

Summary

The a.out broke our build.

Diff Detail

Event Timeline

oontvoo created this revision.Mar 24 2021, 8:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 24 2021, 8:50 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
oontvoo requested review of this revision.Mar 24 2021, 8:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 24 2021, 8:50 AM
gribozavr2 accepted this revision.Mar 24 2021, 8:50 AM
This revision is now accepted and ready to land.Mar 24 2021, 8:50 AM
int3 added inline comments.Mar 24 2021, 12:09 PM
lld/test/MachO/dependency-info.s
23

why doesn't this have the input-file: prefix?

28

for consistency with the input-file lines

oontvoo updated this revision to Diff 333098.Mar 24 2021, 12:55 PM
oontvoo marked 2 inline comments as done.

updated diff

lld/test/MachO/dependency-info.s
23

D'oh - editorial error. (I must've accidentally deleted the prefix at some point) ... of course FileCheck didn't care as long as it matches partially.

int3 accepted this revision.Mar 24 2021, 1:54 PM

lgtm

This revision was landed with ongoing or failed builds.Mar 24 2021, 1:56 PM
This revision was automatically updated to reflect the committed changes.