This is an archive of the discontinued LLVM Phabricator instance.

[lld-macho][nfc] Remove unnecessary -pie flags in tests
ClosedPublic

Authored by int3 on Nov 3 2021, 8:35 AM.

Details

Summary

D101513 means that we no longer need to specify -pie in most of our
test RUN commands. Let's clean up the unused flags so as not to confuse
future test writers.

Diff Detail

Event Timeline

int3 created this revision.Nov 3 2021, 8:35 AM
Herald added a project: Restricted Project. · View Herald Transcript
int3 requested review of this revision.Nov 3 2021, 8:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 3 2021, 8:35 AM
oontvoo accepted this revision.Nov 3 2021, 8:40 AM
oontvoo added a subscriber: oontvoo.

Thanks for the cleanup! :)

lld/test/MachO/tlv.s
6

don't we need a space here?
(same with others -no-pie arg)

This revision is now accepted and ready to land.Nov 3 2021, 8:40 AM
oontvoo added inline comments.Nov 3 2021, 8:41 AM
lld/test/MachO/tlv.s
6

Bleh - this is not the arg - it's the file name.
Ignore me!

MaskRay accepted this revision.Nov 3 2021, 1:46 PM
This revision was automatically updated to reflect the committed changes.