[Mesa-dev] [PATCH 1/3] android: radeonsi: add nir include paths
Emil Velikov
emil.l.velikov at gmail.com
Thu Aug 3 09:57:02 UTC 2017
Hi Mauro,
Thanks for the series. I'll pull 2&3 in a second - there's a minor
suggestion in this patch.
On 3 August 2017 at 01:55, Mauro Rossi <issor.oruam at gmail.com> wrote:
> Android build changes to avoid the following building error:
>
> target C: libmesa_pipe_radeonsi <= external/mesa/src/gallium/drivers/radeonsi/si_pipe.c
> ...
> In file included from external/mesa/src/gallium/drivers/radeonsi/si_pipe.c:38:
> external/mesa/src/compiler/nir/nir.h:48:10: fatal error: 'nir_opcodes.h' file not found
> #include "nir_opcodes.h"
> ^
> 1 error generated.
>
> Fixes: da62a31c5b "radeonsi: add nir include paths"
> ---
> src/gallium/drivers/radeonsi/Android.mk | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/Android.mk b/src/gallium/drivers/radeonsi/Android.mk
> index 6fff91f6f7..452bba3af8 100644
> --- a/src/gallium/drivers/radeonsi/Android.mk
> +++ b/src/gallium/drivers/radeonsi/Android.mk
> @@ -36,7 +36,8 @@ LOCAL_MODULE_CLASS := STATIC_LIBRARIES
>
> LOCAL_C_INCLUDES := \
> $(MESA_TOP)/src/amd/common \
> - $(call generated-sources-dir-for,STATIC_LIBRARIES,libmesa_amd_common,,)/common
> + $(call generated-sources-dir-for,STATIC_LIBRARIES,libmesa_amd_common,,)/common \
> + $(call generated-sources-dir-for,STATIC_LIBRARIES,libmesa_nir,,)/nir
The more robust solution is to add LOCAL_EXPORT_C_INCLUDE_DIRS for libmesa_nir.
With that in place we can drop the existing four instances of the
$(call generated-sources-dir-for... libmesa_nir... from the codebase.
-Emil
More information about the mesa-dev
mailing list