Nice to see this land!
Currently, doesn't LLD/ELF ignore -plugin-opt? That will mean that if a
user uses the "gold syntax" then LLD will silently ignore it, which isn't
good. At the very least, can we issue an error if we see `-plugin-opt
jobs=N` and suggest the LLD spelling?
Or maybe just accept the gold syntax? Our current handling of `-plugin` and
`-plugin-opt` is intended to make LLD transparently Do The Right Thing when
LLD is invoked as if it were gold, so clearly gold compatibility is
important enough for that. This suggests it is important enough to be
compatible from a ThinLTO perspective too.
-- Sean Silva
On Thu, Oct 13, 2016 at 10:42 AM, Davide Italiano via cfe-commits <
> Author: davide
> Date: Thu Oct 13 12:42:38 2016
> New Revision: 284137
> URL: http://llvm.org/viewvc/llvm-project?rev=284137&view=rev
> [ThinLTO] Update doc to include lld (now supported).
> Differential Revision: https://reviews.llvm.org/D25537
> Modified: cfe/trunk/docs/ThinLTO.rst
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/
> --- cfe/trunk/docs/ThinLTO.rst (original)
> +++ cfe/trunk/docs/ThinLTO.rst Thu Oct 13 12:42:38 2016
> @@ -62,8 +62,8 @@ ThinLTO is currently supported for the f
> - **ld64**:
> Starting with `Xcode 8 <https://developer.apple.com/xcode/>`_.
> -Additionally, support is being added to the *lld* linker.
> +- **lld**:
> + Starting with r284050 (ELF only).
> @@ -109,6 +109,8 @@ be reduced to ``N`` via:
> - ld64:
> +- lld:
> + ``-Wl,--thinlto-jobs=N``
> cfe-commits mailing list
cfe-commits mailing list