diff mbox series

[2/2] classes/toolchain/clang: depend on lld-native if ld-is-lld is set

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

Commit Message

Ross Burton July 22, 2025, 12:50 p.m. UTC
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(-)

Comments

Quentin Schulz July 22, 2025, 12:59 p.m. UTC | #1
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
diff mbox series

Patch

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)}"