[PATCH] D53195: [MinGW] Allow using LTO when lld is used as linker

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 12 11:34:14 PDT 2018


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm



================
Comment at: lib/Driver/ToolChains/MinGW.cpp:383
+      Args.getLastArgValue(options::OPT_fuse_ld_EQ, CLANG_DEFAULT_LINKER)
+          .equals_lower("lld");
 }
----------------
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.


https://reviews.llvm.org/D53195





More information about the cfe-commits mailing list