gallium: remove PIPE_SHADER_CAP_OUTPUT_READ
authorMarek Olšák <maraeo@gmail.com>
Thu, 23 Feb 2012 22:44:36 +0000 (23:44 +0100)
committerMarek Olšák <maraeo@gmail.com>
Mon, 27 Feb 2012 01:03:23 +0000 (02:03 +0100)
commit04e324008759282728a95a1394bac2c4c2a1a3f9
tree71691cd97f13431ca8ddb7c902638e5b775eac67
parent74d303521e6ba41d1cbeb75edb2f834ebbe8c550
gallium: remove PIPE_SHADER_CAP_OUTPUT_READ

r600g is the only driver which has made use of it. The reason the CAP was
added was to fix some piglit tests when the GLSL pass lower_output_reads
didn't exist.

However, not removing output reads breaks the fallback for glClampColorARB,
which assumes outputs are not readable. The fix would be non-trivial
and my personal preference is to remove the CAP, considering that reading
outputs is uncommon and that we can now use lower_output_reads to fix
the issue that the CAP was supposed to workaround in the first place.
src/gallium/drivers/i915/i915_screen.c
src/gallium/drivers/nv50/nv50_screen.c
src/gallium/drivers/nvc0/nvc0_screen.c
src/gallium/drivers/r300/r300_screen.c
src/gallium/drivers/r600/r600_pipe.c
src/gallium/drivers/svga/svga_screen.c
src/gallium/include/pipe/p_defines.h
src/mesa/state_tracker/st_glsl_to_tgsi.cpp