mstorsjo added inline comments.
================
Comment at: lib/Driver/ToolChains/MinGW.cpp:383
+ Args.getLastArgValue(options::OPT_fuse_ld_EQ, CLANG_DEFAULT_LINKER)
+ .equals_lower("lld");
}
----------------
rnk wrote:
> I thought we supported putting paths in this option, but I see we already use
> this pattern almost everywhere throughout the driver, so I guess checking for
> the symbolic name is fine.
Well, in ToolChain::GetLinkerPath(), we do support both an absolute path and a
symbolic name, but yes, there are other checks for symbolic names as well. Not
sure if we should try to guess the kind of linker from an absolute pathname
though...
https://reviews.llvm.org/D53195
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
