[PATCH] D13933: [COFF] Add some ld Aliases for gnu mode

Martell Malone via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 21 13:40:29 PDT 2015


martell added a comment.

In http://reviews.llvm.org/D13933#272449, @ruiu wrote:

> If I remember correctly, I recommended to write a shim to convert GNU-style options to link.exe-style options. Does that approach make sense?


Yes and there was objections to this because it adds extra dependencies such as python or whatever language the shim is written in.
I'd rather not have an external dependency / external shim when we could just add support for it directly in the driver which is a lot less work.

Originally compnerd suggested having a conversion system where ld syntax was the end result for every linker but you said that would be too much work or overhaul.

I suggested having a seperate driver based on the ELF driver for this but you weren't excited about the idea of adding a new driver.

rnk then gave his input saying that based on his experience with clang-cl that we should go with the alias approach as it would be the best and have the least impact on the code base.

This is where I thought we were at so i implemented this base patch based on this concept.


http://reviews.llvm.org/D13933





More information about the llvm-commits mailing list