[Mesa-dev] [PATCH mesa] spirv: add missing include path

Ian Romanick idr at freedesktop.org
Tue Jul 18 18:15:49 UTC 2017


I can believe the fix, but I don't understand why it's necessary.
spirv_info.c includes spriv_info.h in the same manner as before
2dd4e2ec, and, as far as I can understand, spirv_info.c gets built in
the same way as before.  Can someone explain to me why it broke?
...because I don't want to break it again.

On 07/18/2017 11:03 AM, Eric Engestrom wrote:
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101831
> Signed-off-by: Eric Engestrom <eric.engestrom at imgtec.com>
> ---
> 
> Note: Android and SCons probably need a similar fix
> ---
>  src/compiler/Makefile.nir.am | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/compiler/Makefile.nir.am b/src/compiler/Makefile.nir.am
> index 1533ee536d..002b9fc535 100644
> --- a/src/compiler/Makefile.nir.am
> +++ b/src/compiler/Makefile.nir.am
> @@ -32,6 +32,9 @@ nir_libnir_la_SOURCES =					\
>  	$(SPIRV_GENERATED_FILES) 			\
>  	$(NIR_GENERATED_FILES)
>  
> +nir_libnir_la_SOURCES = \
> +	-I$(top_builddir)/src/compiler/spirv
> +
>  nir/nir_builder_opcodes.h: nir/nir_opcodes.py nir/nir_builder_opcodes_h.py
>  	$(MKDIR_GEN)
>  	$(PYTHON_GEN) $(srcdir)/nir/nir_builder_opcodes_h.py > $@ || ($(RM) $@; false)
> 



More information about the mesa-dev mailing list