Re: [Mesa-dev] [PATCH 1/7] radv: only display options that are enabled

2018-03-15 Thread Samuel Pitoiset
On 03/15/2018 05:06 PM, Bas Nieuwenhuizen wrote: One comment on patch 7, and the LLVM dump ordering issue we debugged earlier (where we should dump before compiling), otherwise the series is Yes, v2 was here: https://cgit.freedesktop.org/~hakzsam/mesa/commit/?h=radv_llvm_ir&id=904e9e336cbe94

Re: [Mesa-dev] [PATCH 1/7] radv: only display options that are enabled

2018-03-15 Thread Bas Nieuwenhuizen
One comment on patch 7, and the LLVM dump ordering issue we debugged earlier (where we should dump before compiling), otherwise the series is Reviewed-by: Bas Nieuwenhuizen On Wed, Mar 14, 2018 at 12:27 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_de

[Mesa-dev] [PATCH 1/7] radv: only display options that are enabled

2018-03-14 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_debug.c | 28 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/src/amd/vulkan/radv_debug.c b/src/amd/vulkan/radv_debug.c index c8794d06ea..f3b79004f6 100644 --- a/src/amd/vulkan/radv_debug.c +++ b/s