Re: [Mesa-dev] [PATCH] amd/common: Fix build with new ac_add_function_attr()

2017-03-01 Thread Tobias Klausmann
This patch was never intended to fix the warnings, only the build error. If you care to enough to fix the rest, patches are always welcome! Greetings, Tobias On 01.03.17 23:55, Mike Lothian wrote: I can confirm this fixes the build for me But I still get warnings: /var/tmp/portage/media-l

Re: [Mesa-dev] [PATCH] amd/common: Fix build with new ac_add_function_attr()

2017-03-01 Thread Bas Nieuwenhuizen
Pushed, thanks. On Wed, Mar 1, 2017 at 10:16 PM, Tobias Klausmann wrote: > Fix usage of ac_add_function_attr() and make it known! > > common/ac_nir_to_llvm.c: In function 'create_llvm_function': > common/ac_nir_to_llvm.c:265:4: error: implicit declaration of function > 'ac_add_function_attr' [-We

Re: [Mesa-dev] [PATCH] amd/common: Fix build with new ac_add_function_attr()

2017-03-01 Thread Mike Lothian
I can confirm this fixes the build for me But I still get warnings: /var/tmp/portage/media-libs/mesa-/work/mesa-/src/amd/common/ ac_nir_to_llvm.c: In function ‘visit_interp’: /var/tmp/portage/media-libs/mesa-/work/mesa-/src/amd/common/ ac_nir_to_llvm.c:3022:11: warning: ‘location’

[Mesa-dev] [PATCH] amd/common: Fix build with new ac_add_function_attr()

2017-03-01 Thread Tobias Klausmann
Fix usage of ac_add_function_attr() and make it known! common/ac_nir_to_llvm.c: In function 'create_llvm_function': common/ac_nir_to_llvm.c:265:4: error: implicit declaration of function 'ac_add_function_attr' [-Werror=implicit-function-declaration] ac_add_function_attr(main_function, i + 1, A