Am Mittwoch, 12. Oktober 2016, 09:52:51 CEST schrieb Emil Velikov:
> On 12 October 2016 at 00:02, Tobias Droste <tdro...@gmx.de> wrote:
> > Add functions to add and check targets/components.
> > Not used in this patch.
> >
> > The error message in llvm_add_component is disabled until it doesn't break
> > the build anymore. This is the same functionality as before where the
> > components were added without a check.
> >
> > Signed-off-by: Tobias Droste <tdro...@gmx.de>
> > ---
> >
> > configure.ac | 35 +++++++++++++++++++++++++++++++++++
> > 1 file changed, 35 insertions(+)
> >
> > diff --git a/configure.ac b/configure.ac
> > index bdd46bc..69421ff 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -2196,7 +2196,42 @@ llvm_check_version_for() {
> >
> > fi
> >
> > }
> >
> > +llvm_add_default_components() {
> > + driver_name=$1
> >
> > + # Required default components
> > + llvm_add_component "engine" $driver_name
> > + llvm_add_component "bitwriter" $driver_name
> > + llvm_add_component "mcjit" $driver_name
> > + llvm_add_component "mcdisassembler" $driver_name
> > +
> > + # Optional default components
> > + if $LLVM_CONFIG --components | grep -iqw inteljitevents ; then
> > + LLVM_COMPONENTS="${LLVM_COMPONENTS} inteljitevents"
> > + fi
> > +}
> > +
> > +llvm_add_component() {
>
> Nit: move this function before its user (llvm_add_default_components)
> Idea: call this ...components (plural) and feed the all components at once.
Ok.
>
> > + new_llvm_component=$1
> > + driver_name=$2
> > +
> > + if $LLVM_CONFIG --components | grep -iqw $new_llvm_component ; then
> > + LLVM_COMPONENTS="${LLVM_COMPONENTS} ${new_llvm_component}"
> > + #else
> > + # AC_MSG_ERROR([LLVM component '$new_llvm_component' not enabled
> > in your LLVM build. Required by $driver_name.])
> This function adds required components, correct ? Then the above two
> lines should not be commented out.
I can't enable this until later in the series as mentioned in the commit
message.
This is how it worked before (there were no errors for the components).
At this point llvm_add_default_components will be added even if we don't need
LLVM. This way someone without LLVM installed will get error messages even
though he might just want to build the intel or softpipe driver.
It will be enabled in Patch 24.
>
> -Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
