diff mbox series

[2/6] clang: disable all test targets

Message ID 20250912143038.3345369-2-ross.burton@arm.com
State Accepted, archived
Commit ba358a03c97cb904248b743c483a7b3f4b9159a9
Headers show
Series [1/6] clang: improve opt-viewer PACKAGECONFIG | expand

Commit Message

Ross Burton Sept. 12, 2025, 2:30 p.m. UTC
We don't install the tests, or run them in the build tree, so tell cmake
not to generate the targets at all. This removes internal dependencies
which means we can turn off more components that we don't want.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 meta/recipes-devtools/clang/clang_git.bb | 1 +
 1 file changed, 1 insertion(+)

Comments

Khem Raj Sept. 15, 2025, 5:14 a.m. UTC | #1
On Fri, Sep 12, 2025 at 7:30 AM Ross Burton via lists.openembedded.org
<ross.burton=arm.com@lists.openembedded.org> wrote:
>
> We don't install the tests, or run them in the build tree, so tell cmake
> not to generate the targets at all. This removes internal dependencies
> which means we can turn off more components that we don't want.
>

Hmm, there is a plan to enable running ptest support to clang, it will be good
to keep that in mind while we add this, so we do not have to redo things again.

> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---
>  meta/recipes-devtools/clang/clang_git.bb | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-devtools/clang/clang_git.bb b/meta/recipes-devtools/clang/clang_git.bb
> index f53ecbfe398..09dda563bd3 100644
> --- a/meta/recipes-devtools/clang/clang_git.bb
> +++ b/meta/recipes-devtools/clang/clang_git.bb
> @@ -120,6 +120,7 @@ EXTRA_OECMAKE += "-DLLVM_ENABLE_ASSERTIONS=OFF \
>                    -DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD='${LLVM_EXPERIMENTAL_TARGETS_TO_BUILD}' \
>                    -DLLVM_NATIVE_TOOL_DIR=${STAGING_BINDIR_NATIVE} \
>                    -DLLVM_HEADERS_TABLEGEN=${STAGING_BINDIR_NATIVE}/llvm-min-tblgen \
> +                  -DLLVM_INCLUDE_TESTS=OFF \
>  "
>
>  EXTRA_OECMAKE:append:class-nativesdk = "\
> --
> 2.43.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#223314): https://lists.openembedded.org/g/openembedded-core/message/223314
> Mute This Topic: https://lists.openembedded.org/mt/115206857/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Ross Burton Sept. 15, 2025, 9:39 a.m. UTC | #2
On 15 Sep 2025, at 06:14, Khem Raj <raj.khem@gmail.com> wrote:
> 
> On Fri, Sep 12, 2025 at 7:30 AM Ross Burton via lists.openembedded.org
> <ross.burton=arm.com@lists.openembedded.org> wrote:
>> 
>> We don't install the tests, or run them in the build tree, so tell cmake
>> not to generate the targets at all. This removes internal dependencies
>> which means we can turn off more components that we don't want.
>> 
> 
> Hmm, there is a plan to enable running ptest support to clang, it will be good
> to keep that in mind while we add this, so we do not have to redo things again.

If that happens, all that needs to happen is turning the -DLLVM_INCLUDE_TESTS=OFF into a PACKAGECONFIG.

Ross
diff mbox series

Patch

diff --git a/meta/recipes-devtools/clang/clang_git.bb b/meta/recipes-devtools/clang/clang_git.bb
index f53ecbfe398..09dda563bd3 100644
--- a/meta/recipes-devtools/clang/clang_git.bb
+++ b/meta/recipes-devtools/clang/clang_git.bb
@@ -120,6 +120,7 @@  EXTRA_OECMAKE += "-DLLVM_ENABLE_ASSERTIONS=OFF \
                   -DLLVM_EXPERIMENTAL_TARGETS_TO_BUILD='${LLVM_EXPERIMENTAL_TARGETS_TO_BUILD}' \
                   -DLLVM_NATIVE_TOOL_DIR=${STAGING_BINDIR_NATIVE} \
                   -DLLVM_HEADERS_TABLEGEN=${STAGING_BINDIR_NATIVE}/llvm-min-tblgen \
+                  -DLLVM_INCLUDE_TESTS=OFF \
 "
 
 EXTRA_OECMAKE:append:class-nativesdk = "\