This is an archive of the discontinued LLVM Phabricator instance.

Update hexagon driver tests
ClosedPublic

Authored by sidneym on Oct 12 2018, 2:28 PM.

Details

Summary

Don't assume the linker is hexagon-link. https://reviews.llvm.org/D53038 removed the hard-coded name so now it is possible for the other named linkers to be used.

Assume either hexagon-link or *ld*.

Diff Detail

Repository
rL LLVM

Event Timeline

sidneym created this revision.Oct 12 2018, 2:28 PM

Tested and verified with -DCLANG_DEFAULT_LINKER=lld

davezarzycki accepted this revision.Oct 12 2018, 3:34 PM
This revision is now accepted and ready to land.Oct 12 2018, 3:34 PM
This revision was automatically updated to reflect the committed changes.
phosek added a subscriber: phosek.Oct 14 2018, 1:59 PM

The way we addressed this for other targets in the past is to explicitly set -fuse-ld=ld for all tests, I made that change in D53249.