[PATCH] D42673: [RISCV] Pick the correct RISCV linker instead of calling riscv-gcc to link

2018-02-01 Thread Mandeep Singh Grang via Phabricator via cfe-commits
mgrang abandoned this revision. mgrang added a comment. For now, by setting the full triple (riscv-unknown-linux-gnu) we are able to invoke the correct gnu linker. So will abandon this patch. Repository: rC Clang https://reviews.llvm.org/D42673

[PATCH] D42673: [RISCV] Pick the correct RISCV linker instead of calling riscv-gcc to link

2018-01-31 Thread Mandeep Singh Grang via Phabricator via cfe-commits
mgrang added a comment. In https://reviews.llvm.org/D42673#992056, @asb wrote: > My main concern with this patch is that the description doesn't really match > what it does. The current in-tree code _doesn't_ call gcc to link for the > tested configuration (a multilib toolchain), and this is

[PATCH] D42673: [RISCV] Pick the correct RISCV linker instead of calling riscv-gcc to link

2018-01-30 Thread Alex Bradbury via Phabricator via cfe-commits
asb added a comment. My main concern with this patch is that the description doesn't really match what it does. The current in-tree code _doesn't_ call gcc to link for the tested configuration (a multilib toolchain), and this is verified with the tests in test/Driver/riscv32-toolchain.c.

[PATCH] D42673: [RISCV] Pick the correct RISCV linker instead of calling riscv-gcc to link

2018-01-29 Thread Mandeep Singh Grang via Phabricator via cfe-commits
mgrang created this revision. mgrang added reviewers: asb, apazos. mgrang added a project: clang. Herald added subscribers: cfe-commits, hintonda, niosHD, sabuasal, jordy.potman.lists, simoncook, johnrusso, rbar, mgorny. Currently we are invoking riscv-gcc to link. This causes all flags passed