Am 10.08.2012 03:42, schrieb Ian Romanick: > On 08/08/2012 10:53 AM, Brian Paul wrote: >> On 08/08/2012 11:38 AM, Ian Romanick wrote: >>> From: Ian Romanick<ian.d.roman...@intel.com> >>> >>> Signed-off-by: Ian Romanick<ian.d.roman...@intel.com> >>> --- >>> src/mesa/main/extensions.c | 24 ++++++++++++------------ >>> 1 files changed, 12 insertions(+), 12 deletions(-) >>> >>> diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c >>> index 5115d42..cca1eac 100644 >>> --- a/src/mesa/main/extensions.c >>> +++ b/src/mesa/main/extensions.c >>> @@ -88,7 +88,7 @@ static const struct extension extension_table[] = { >>> { "GL_ARB_debug_output", >>> o(dummy_true), GL, 2009 }, >>> { "GL_ARB_depth_buffer_float", >>> o(ARB_depth_buffer_float), GL, 2008 }, >>> { "GL_ARB_depth_clamp", >>> o(ARB_depth_clamp), GL, 2003 }, >>> - { "GL_ARB_depth_texture", >>> o(ARB_depth_texture), GL, 2001 }, >>> + { "GL_ARB_depth_texture", >>> o(ARB_depth_texture), GLL, 2001 }, >>> { "GL_ARB_draw_buffers", >>> o(dummy_true), GL, 2002 }, >>> { "GL_ARB_draw_buffers_blend", >>> o(ARB_draw_buffers_blend), GL, 2009 }, >>> { "GL_ARB_draw_elements_base_vertex", >>> o(ARB_draw_elements_base_vertex), GL, 2009 }, >>> @@ -109,7 +109,7 @@ static const struct extension extension_table[] = { >>> { "GL_ARB_occlusion_query2", >>> o(ARB_occlusion_query2), GL, 2003 }, >>> { "GL_ARB_occlusion_query", >>> o(ARB_occlusion_query), GL, 2001 }, >>> { "GL_ARB_pixel_buffer_object", >>> o(EXT_pixel_buffer_object), GL, 2004 }, >>> - { "GL_ARB_point_parameters", >>> o(EXT_point_parameters), GL, 1997 }, >>> + { "GL_ARB_point_parameters", >>> o(EXT_point_parameters), GLL, 1997 }, >>> { "GL_ARB_point_sprite", >>> o(ARB_point_sprite), GL, 2003 }, >>> { "GL_ARB_provoking_vertex", >>> o(EXT_provoking_vertex), GL, 2009 }, >>> { "GL_ARB_robustness", >>> o(dummy_true), GL, 2010 }, >>> @@ -121,7 +121,7 @@ static const struct extension extension_table[] = { >>> { "GL_ARB_shader_texture_lod", >>> o(ARB_shader_texture_lod), GL, 2009 }, >>> { "GL_ARB_shading_language_100", >>> o(ARB_shading_language_100), GLL, 2003 }, >>> { "GL_ARB_shadow_ambient", >>> o(ARB_shadow_ambient), GL, 2001 }, >> >> I think GL_ARB_shadow_ambient would go hand-in-hand with GL_ARB_shadow. > > I didn't include that extension because it never became part of any GL > version. It's just an extension. I believe that Radeon hardware > supports this natively. Everyone else would need to add a MAX > instruction in the shader, so nobody ever bothered. Realistically, it's > only useful with fixed function. Well certainly not r100/r200 (not supporting arb_depth_texture/arb_shadow is the reason r200 is only 1.3). r300 doesn't support arb_shadow neither (emulated in the shader so I guess in a way you could say shadow_ambient is supported natively :-)). You can't actually emulate shadow_ambient in the shader easily, since all modern hw (afaik) does the suggested per-sample (instead of per-pixel) comparison (ok should still be possible to emulate but it's slightly more complicated).
Roland > > I don't have a strong opinion either way. > >> Otherwise patches 1-6 look good to me. We'll probably make more tweaks >> in the future. >> >> Reviewed-by: Brian Paul <bri...@vmware.com> > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev