| Message ID | 20250520110111.3893998-1-dmitry.baryshkov@oss.qualcomm.com |
|---|---|
| State | Accepted, archived |
| Commit | 177aaa7912f317da4a17a57081eb4f5667ef2c02 |
| Headers | show |
| Series | clang: build-depend on spirv-llvm-translator-native | expand |
On 5/20/25 4:01 AM, Dmitry Baryshkov via lists.openembedded.org wrote: > Building of spirv-mesa and spirv64-mesa CLC targets (which are required > for Mesa to work) requires a working llvm-spirv tool (provided by the > spirv-llvm-translator-native package). Make clang build-depend on the > corresponding recipe in order to be able to build requried targets. > > Fixes: 4178fe97371b ("clang: split SPIRV-LLVM-Translator to its own recipe") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > --- > meta/recipes-devtools/clang/clang_git.bb | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meta/recipes-devtools/clang/clang_git.bb b/meta/recipes-devtools/clang/clang_git.bb > index b41b700292c7..bca8c7fa877d 100644 > --- a/meta/recipes-devtools/clang/clang_git.bb > +++ b/meta/recipes-devtools/clang/clang_git.bb > @@ -206,7 +206,7 @@ EXTRA_OECMAKE:append:class-target = "\ > > DEPENDS = "binutils zlib zstd libffi libxml2 libxml2-native ninja-native swig-native spirv-tools-native" > DEPENDS:append:class-nativesdk = " clang-crosssdk-${SDK_SYS} virtual/nativesdk-cross-binutils nativesdk-python3" > -DEPENDS:append:class-target = " clang-cross-${TARGET_ARCH} python3 ${@bb.utils.contains('TC_CXX_RUNTIME', 'llvm', 'compiler-rt libcxx', '', d)}" > +DEPENDS:append:class-target = " clang-cross-${TARGET_ARCH} python3 ${@bb.utils.contains('TC_CXX_RUNTIME', 'llvm', 'compiler-rt libcxx', '', d)} spirv-llvm-translator-native" LGTM > > RRECOMMENDS:${PN} = "binutils" > RRECOMMENDS:${PN}:append:class-target = "${@bb.utils.contains('TC_CXX_RUNTIME', 'llvm', ' libcxx-dev', '', d)}" > > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#216902): https://lists.openembedded.org/g/openembedded-core/message/216902 > Mute This Topic: https://lists.openembedded.org/mt/113208799/1997914 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- >
diff --git a/meta/recipes-devtools/clang/clang_git.bb b/meta/recipes-devtools/clang/clang_git.bb index b41b700292c7..bca8c7fa877d 100644 --- a/meta/recipes-devtools/clang/clang_git.bb +++ b/meta/recipes-devtools/clang/clang_git.bb @@ -206,7 +206,7 @@ EXTRA_OECMAKE:append:class-target = "\ DEPENDS = "binutils zlib zstd libffi libxml2 libxml2-native ninja-native swig-native spirv-tools-native" DEPENDS:append:class-nativesdk = " clang-crosssdk-${SDK_SYS} virtual/nativesdk-cross-binutils nativesdk-python3" -DEPENDS:append:class-target = " clang-cross-${TARGET_ARCH} python3 ${@bb.utils.contains('TC_CXX_RUNTIME', 'llvm', 'compiler-rt libcxx', '', d)}" +DEPENDS:append:class-target = " clang-cross-${TARGET_ARCH} python3 ${@bb.utils.contains('TC_CXX_RUNTIME', 'llvm', 'compiler-rt libcxx', '', d)} spirv-llvm-translator-native" RRECOMMENDS:${PN} = "binutils" RRECOMMENDS:${PN}:append:class-target = "${@bb.utils.contains('TC_CXX_RUNTIME', 'llvm', ' libcxx-dev', '', d)}"
Building of spirv-mesa and spirv64-mesa CLC targets (which are required for Mesa to work) requires a working llvm-spirv tool (provided by the spirv-llvm-translator-native package). Make clang build-depend on the corresponding recipe in order to be able to build requried targets. Fixes: 4178fe97371b ("clang: split SPIRV-LLVM-Translator to its own recipe") Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> --- meta/recipes-devtools/clang/clang_git.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)