Just a message to Eric,
as per our previous private thread,
I've checked that the Android build works,
but we use libLLVM70 name in library dependency.
Please adapt and apply the patch to mesa dev branch,
to fix the breakage
\ prior to " in the LLVM version string value, just touching
Android.mk solves the problem for me
Mauro
On Sat, Apr 13, 2019 at 1:27 AM Mauro Rossi <issor.or...@gmail.com> wrote:
>
> Fixes the following building errors:
>
> external/mesa/src/gallium/drivers/r600/r600_pipe_common.c:1290:14:
> error: expected ')'
> ", LLVM " MESA_LLVM_VERSION_STRING
> ^
> <command line>:8:34: note: expanded from here
> ^
> external/mesa/src/gallium/drivers/r600/r600_pipe_common.c:1287:10:
> note: to match this '('
> snprintf(rscreen->renderer_string, sizeof(rscreen->renderer_string),
> ^
> 1 error generated.
>
> Fixes: 05b114e ("simplify LLVM version string printing")
> Signed-off-by: Mauro Rossi <issor.or...@gmail.com>
> ---
> Android.mk | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/Android.mk b/Android.mk
> index 09139e86d1..b835eb64e9 100644
> --- a/Android.mk
> +++ b/Android.mk
> @@ -97,13 +97,13 @@ define mesa-build-with-llvm
> $(if $(filter $(MESA_ANDROID_MAJOR_VERSION), 4 5), \
> $(warning Unsupported LLVM version in Android
> $(MESA_ANDROID_MAJOR_VERSION)),) \
> $(if $(filter 6,$(MESA_ANDROID_MAJOR_VERSION)), \
> - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0307
> -DMESA_LLVM_VERSION_STRING="3.7")) \
> + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0307
> -DMESA_LLVM_VERSION_STRING=\"3.7\")) \
> $(if $(filter 7,$(MESA_ANDROID_MAJOR_VERSION)), \
> - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> -DMESA_LLVM_VERSION_STRING="7.0")) \
> + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> -DMESA_LLVM_VERSION_STRING=\"7.0\")) \
> $(if $(filter 8,$(MESA_ANDROID_MAJOR_VERSION)), \
> - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> -DMESA_LLVM_VERSION_STRING="7.0")) \
> + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0700
> -DMESA_LLVM_VERSION_STRING=\"7.0\")) \
> $(if $(filter 9,$(MESA_ANDROID_MAJOR_VERSION)), \
> - $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0309
> -DMESA_LLVM_VERSION_STRING="3.9")) \
> + $(eval LOCAL_CFLAGS += -DHAVE_LLVM=0x0309
> -DMESA_LLVM_VERSION_STRING=\"3.9\")) \
> $(eval LOCAL_SHARED_LIBRARIES += libLLVM70)
> endef
>
> --
> 2.20.1
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
