[Mesa-dev] [PATCH 2/8] i965: Add INTEL_DEBUG=perf for failure to compile 16-wide shaders.

Eric Anholt eric at anholt.net
Sun Aug 12 19:05:47 PDT 2012


Jordan Justen <jordan.l.justen at intel.com> writes:

> On Tue, 2012-08-07 at 11:04 -0700, Eric Anholt wrote:
>> diff --git a/src/mesa/drivers/dri/intel/intel_context.h b/src/mesa/drivers/dri/intel/intel_context.h
>> index 6d1a81c..c4efa54 100644
>> --- a/src/mesa/drivers/dri/intel/intel_context.h
>> +++ b/src/mesa/drivers/dri/intel/intel_context.h
>> @@ -463,6 +463,11 @@ extern int INTEL_DEBUG;
>>  		printf(__VA_ARGS__);				\
>>  } while(0)
>>  
>> +#define perf_debug(...) do {					\
>> +	if (unlikely(INTEL_DEBUG & DEBUG_PERF))			\
>> +		printf(__VA_ARGS__);				\
>> +} while(0)
>
> Should perf_debug be used in the paths in PATCH 1/8?
>
> For series:
> Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>

Seems like a reasonable follow-up, but I don't want to block on it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20120812/c9029223/attachment.pgp>


More information about the mesa-dev mailing list