[Mesa-stable] [PATCH] opencl: autotools: Fix linking order for OpenCL target

[Aaron Watry]

Given the discussion that's ongoing, this patch might not land as-is,
but if it does:
Tested-By: Aaron Watry <awatry at gmail.com>

Note: The meson build currently works as-is and doesn't require an
equivalent patch.

--Aaron

On Tue, May 1, 2018 at 7:14 AM, Kai Wasserbäch
<kai at dev.carbon-project.org> wrote:
> Otherwise the build fails with an undefined reference to
> clang::FrontendTimesIsEnabled.
>
> Bugzilla: https://bugs.freedesktop.org/106209
> Cc: mesa-stable at lists.freedesktop.org
> Cc: Jan Vesely <jan.vesely at rutgers.edu>
> Signed-off-by: Kai Wasserbäch <kai at dev.carbon-project.org>
> ---
>
> Hey,
> this patch fixes a FTBFS for me with recent LLVM/Clang 7 revisions from
> upstream's SVN (I use the packages from apt.llvm.org).
>
> If you accept it, please commit it for me, I do not have commit access.
>
> The CC to stable can be dropped, if stable branches are not to be
> expected to be buildable with LLVM/Clang from SVN.
>
> Thank you in advance for considering this patch.
>
> Cheers,
> Kai
>
>
>  src/gallium/targets/opencl/Makefile.am | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/gallium/targets/opencl/Makefile.am b/src/gallium/targets/opencl/Makefile.am
> index de68a93ad5..f0e1de7797 100644
> --- a/src/gallium/targets/opencl/Makefile.am
> +++ b/src/gallium/targets/opencl/Makefile.am
> @@ -23,11 +23,10 @@ lib at OPENCL_LIBNAME@_la_LIBADD = \
>         $(LIBELF_LIBS) \
>         $(DLOPEN_LIBS) \
>         -lclangCodeGen \
> -       -lclangFrontendTool \
>         -lclangFrontend \
> +       -lclangFrontendTool \
>         -lclangDriver \
>         -lclangSerialization \
> -       -lclangCodeGen \
>         -lclangParse \
>         -lclangSema \
>         -lclangAnalysis \
> --
> 2.17.0
>
> _______________________________________________
> mesa-stable mailing list
> mesa-stable at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-stable