The previous patch fixed the cause of the Piglit failure with ARB_color_buffer_float fragment clamp modes. Now that it's fixed, there's no reason to leave it disabled.
Cc: Eric Anholt <[email protected]> Signed-off-by: Kenneth Graunke <[email protected]> --- src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c index 4718337..9802787 100644 --- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c +++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c @@ -525,14 +525,9 @@ brw_init_surface_formats(struct brw_context *brw) * integer, so we don't need hardware support for blending on it. Other * than that, GL in general requires alpha blending for render targets, * even though we don't support it for some formats. - * - * We don't currently support rendering to SNORM textures because some of - * the ARB_color_buffer_float clamping is broken for it - * (piglit arb_color_buffer_float-drawpixels GL_RGBA8_SNORM). */ if (gen >= rinfo->render_target && - (gen >= rinfo->alpha_blend || is_integer) && - _mesa_get_format_datatype(format) != GL_SIGNED_NORMALIZED) { + (gen >= rinfo->alpha_blend || is_integer)) { brw->render_target_format[format] = render; brw->format_supported_as_render_target[format] = true; } -- 1.7.10.4 _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
