sidneym added a comment.
I updated the tests: https://reviews.llvm.org/D53219
Repository:
rC Clang
https://reviews.llvm.org/D53038
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sidneym added a comment.
Seems like the test cases do not consider the possibility of
CLANG_DEFAULT_LINKER being set. If CLANG_DEFAULT_LINKER was set to
"hexagon-link" these tests would probably pass.
I'm checking this out now.
Repository:
rC Clang
https://reviews.llvm.org/D53038
davezarzycki added a comment.
A bisect revealed this change as the source of test failures on my Red Hat
Fedora 28 Linux box. How was/is this tested?
Command Output (stderr):
/home/dave/s/lc/tools/clang/test/Driver/linux-ld.c:882:19: error:
CHECK-HEXAGON: expected
This revision was automatically updated to reflect the committed changes.
Closed by commit rC344147: [Hexagon] Use GetLinkerPath instead of hard-coded
string. (authored by sidneym, committed by ).
Repository:
rC Clang
https://reviews.llvm.org/D53038
Files:
lib/Driver/ToolChains/Hexagon.cpp
sidneym created this revision.
sidneym added reviewers: shankare, ruiu, kparzysz, bcain.
Herald added a subscriber: cfe-commits.
Use GetLinkerPath method instead of hard-coding linker name.
Change should allow -fuse-ld to work on the Hexagon target.
Repository:
rC Clang
