On Thursday, 2017-10-05 10:19:07 +0000, Emil Velikov wrote:
> From: Emil Velikov <emil.veli...@collabora.com>
>
> We want to add "optional" components, which have been added with later
> LLVM versions.
>
> One such in-tree example is inteljitevents. Others are to follow
> shortly.
>
> Cc: mesa-sta...@lists.freedesktop.org
> Cc: Tobias Droste <tdro...@gmx.de>
> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
> ---
> Tobias, any preference if we keep this separate helperor fold it with
> llvm_add_component?
>
> Should we fold the llvm_add_default_components usecase here, or keep it
> separate patch.
> ---
> configure.ac | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index dbaa6569e4e..56cbf26c778 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -955,6 +955,14 @@ llvm_add_component() {
> fi
> }
>
> +llvm_add_optional_component() {
> + new_llvm_component=$1
> + driver_name=$2
> +
> + if $LLVM_CONFIG --components | grep -iqw $new_llvm_component ; then
> + LLVM_COMPONENTS="${LLVM_COMPONENTS} ${new_llvm_component}"
> + fi
> +}
> llvm_add_default_components() {
> driver_name=$1
>
> @@ -965,9 +973,7 @@ llvm_add_default_components() {
> llvm_add_component "mcjit" $driver_name
>
> # Optional default components
> - if $LLVM_CONFIG --components | grep -iqw inteljitevents ; then
> - LLVM_COMPONENTS="$LLVM_COMPONENTS inteljitevents"
> - fi
> + llvm_add_component "inteljitevents" $driver_name
You meant `llvm_add_optional_component`, right?
> }
>
> llvm_add_target() {
> --
> 2.14.1
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
