Re: [Mesa-dev] [PATCH 1/9] i965: Fix "SIMD16 unsupported" messages via KHR_debug.

2014-04-07 Thread Matt Turner
On Sun, Apr 6, 2014 at 11:49 PM, Kenneth Graunke wrote: > Performance warnings are logged via KHR_debug in addition to when the > INTEL_DEBUG=perf environment variable is set. Without this, messages in > debug contexts would have "(null)" for the reason. > > Signed-off-by: Kenneth Graunke > ---

Re: [Mesa-dev] [PATCH 1/9] i965: Fix "SIMD16 unsupported" messages via KHR_debug.

2014-04-07 Thread Ian Romanick
I made a couple random comments on several patches. I also like Ilia's suggestion on patch 3. Patches 1 through 8 (with Ilia's suggestion on 3) are Reviewed-by: Ian Romanick I think patch 9 still needs some work for signed numerator handling in division. On 04/06/2014 11:49 PM, Kenneth Graunk

[Mesa-dev] [PATCH 1/9] i965: Fix "SIMD16 unsupported" messages via KHR_debug.

2014-04-06 Thread Kenneth Graunke
Performance warnings are logged via KHR_debug in addition to when the INTEL_DEBUG=perf environment variable is set. Without this, messages in debug contexts would have "(null)" for the reason. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +- 1 file changed, 1 inse