[PATCH] D87418: [LLD] Allow configuring default ld.lld backend
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 9 21:01:40 PDT 2020
mstorsjo added reviewers: MaskRay, amccarth, rnk, mstorsjo.
mstorsjo added a comment.
As a concept, this looks sensible to me, similar to how llvm can be configured with a default triple or how clang can have cmake-configurable built-in defaults for linker/stdlib.
================
Comment at: lld/tools/lld/lld.cpp:96
+
+ return LLD_DEFAULT_LD_LLD_BACKEND == "MINGW" ? true : false;
}
----------------
Wrap either side in `StringRef()` to get proper behaviour for the `==` - or maybe evaluate it to an int/bool already in cmake to avoid any actual runtime code for this? (Sorry for not picking up on this before.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87418/new/
https://reviews.llvm.org/D87418
More information about the llvm-commits
mailing list