Re: [Mesa-dev] [PATCH 1/2] radeonsi: Print "LLVM emitted unknown config register" warning only once

2016-01-15 Thread Gustaw Smolarczyk
Hi Michel. What will happen if there are 2 different uknown registers? With this patch, only the first one will be shown. Gustaw Smolarczyk 2016-01-15 10:23 GMT+01:00 : > This series is, > > Reviewed-by: Edward O'Callaghan > > > On

Re: [Mesa-dev] [PATCH 1/2] radeonsi: Print "LLVM emitted unknown config register" warning only once

2016-01-15 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák On Fri, Jan 15, 2016 at 4:23 AM, Michel Dänzer wrote: > From: Michel Dänzer > > Say "LLVM" instead of "Compiler" for clarity. > > Signed-off-by: Michel Dänzer >

Re: [Mesa-dev] [PATCH 1/2] radeonsi: Print "LLVM emitted unknown config register" warning only once

2016-01-15 Thread Nicolai Hähnle
On 15.01.2016 05:30, Gustaw Smolarczyk wrote: Hi Michel. What will happen if there are 2 different uknown registers? With this patch, only the first one will be shown. That doesn't really matter. The point is to give some indication of a possible cause of badness, i.e. a hint that one might

Re: [Mesa-dev] [PATCH 1/2] radeonsi: Print "LLVM emitted unknown config register" warning only once

2016-01-15 Thread eocallaghan
This series is, Reviewed-by: Edward O'Callaghan On 2016-01-15 14:23, Michel Dänzer wrote: From: Michel Dänzer Say "LLVM" instead of "Compiler" for clarity. Signed-off-by: Michel Dänzer ---

[Mesa-dev] [PATCH 1/2] radeonsi: Print "LLVM emitted unknown config register" warning only once

2016-01-14 Thread Michel Dänzer
From: Michel Dänzer Say "LLVM" instead of "Compiler" for clarity. Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeonsi/si_shader.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git