| Message ID | 20250722125041.3301907-2-ross.burton@arm.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/2] clang: split lld into a separate lld recipe | expand |
Hi Ross, On 7/22/25 2:50 PM, Ross Burton via lists.openembedded.org wrote: > As lld is no longer part of the clang recipe, also depend on lld-native > if ld-is-lld has been set. > This should be squashed with the previous commit to prevent breaking bisectability and make it easier to revert the previous commit if we need to? Cheers, Quentin
On Tue, Jul 22, 2025 at 5:50 AM Ross Burton <ross.burton@arm.com> wrote: > > As lld is no longer part of the clang recipe, also depend on lld-native > if ld-is-lld has been set. > > Signed-off-by: Ross Burton <ross.burton@arm.com> > --- > meta/classes/toolchain/clang.bbclass | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meta/classes/toolchain/clang.bbclass b/meta/classes/toolchain/clang.bbclass > index 7c6ed57fb31..76419e510b4 100644 > --- a/meta/classes/toolchain/clang.bbclass > +++ b/meta/classes/toolchain/clang.bbclass > @@ -27,7 +27,7 @@ PREFERRED_PROVIDER_virtual/nativesdk-cross-cc:class-cross-canadian = "clang-cros > PREFERRED_PROVIDER_virtual/nativesdk-cross-c++:class-cross-canadian = "clang-crosssdk-${SDK_SYS}" > > > -BASE_DEFAULT_DEPS:append:class-target = " compiler-rt libcxx" > +BASE_DEFAULT_DEPS:append:class-target = " compiler-rt libcxx ${@bb.utils.contains('DISTRO_FEATURES', 'ld-is-lld', ' lld-native', '', d)}" This would be needed for all clang variants, since -fuse-ld=lld should work for native and nativesdk as well. > > TUNE_CCARGS += "${@bb.utils.contains("DISTRO_FEATURES", "usrmerge", " --dyld-prefix=/usr", "", d)}" > > -- > 2.43.0 >
diff --git a/meta/classes/toolchain/clang.bbclass b/meta/classes/toolchain/clang.bbclass index 7c6ed57fb31..76419e510b4 100644 --- a/meta/classes/toolchain/clang.bbclass +++ b/meta/classes/toolchain/clang.bbclass @@ -27,7 +27,7 @@ PREFERRED_PROVIDER_virtual/nativesdk-cross-cc:class-cross-canadian = "clang-cros PREFERRED_PROVIDER_virtual/nativesdk-cross-c++:class-cross-canadian = "clang-crosssdk-${SDK_SYS}" -BASE_DEFAULT_DEPS:append:class-target = " compiler-rt libcxx" +BASE_DEFAULT_DEPS:append:class-target = " compiler-rt libcxx ${@bb.utils.contains('DISTRO_FEATURES', 'ld-is-lld', ' lld-native', '', d)}" TUNE_CCARGS += "${@bb.utils.contains("DISTRO_FEATURES", "usrmerge", " --dyld-prefix=/usr", "", d)}"
As lld is no longer part of the clang recipe, also depend on lld-native if ld-is-lld has been set. Signed-off-by: Ross Burton <ross.burton@arm.com> --- meta/classes/toolchain/clang.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)