On 08/09/2011 03:33 PM, Ian Romanick wrote:
> On 08/09/2011 10:59 AM, Kenneth Graunke wrote:
>> It's the same as GL_AMD_conservative_depth.  The specs have slight
>> differences in wording, but don't differ in content or behavior.
> 
>> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> 
> Why do you add ARB_conservative_depth_{enable,warn} flags to
> _mesa_glsl_parse_state but not use them?

They are used, just non-obviously.  The EXT macro call below uses them:

>> diff --git a/src/glsl/glsl_parser_extras.cpp 
>> b/src/glsl/glsl_parser_extras.cpp
>> index cc78137..8f740e6 100644
>> --- a/src/glsl/glsl_parser_extras.cpp
>> +++ b/src/glsl/glsl_parser_extras.cpp
>> @@ -253,6 +253,7 @@ struct _mesa_glsl_extension {
>>  static const _mesa_glsl_extension _mesa_glsl_supported_extensions[] = {
>>     /*                                  target availability  API 
>> availability */
>>     /* name                             VS     GS     FS     GL     ES       
>>   supported flag */
>> +   EXT(ARB_conservative_depth,         true,  false, true,  true,  false,   
>>   AMD_conservative_depth),
>>     EXT(ARB_draw_buffers,               false, false, true,  true,  false,   
>>   dummy_true),
>>     EXT(ARB_draw_instanced,             true,  false, false, true,  false,   
>>   ARB_draw_instanced),
>>     EXT(ARB_explicit_attrib_location,   true,  false, true,  true,  false,   
>>   ARB_explicit_attrib_location),
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to