Paul Berry <stereotype...@gmail.com> writes: > Previously, the code for setting this flag for GLSL programs was > duplicated in three places: brw_link_shader(), glsl_to_tgsi_visitor, > and ir_to_mesa_visitor. In addition to the unnecessary duplication, > there was a performance problem on i965: brw_link_shader() set the > flag before doing its final round of optimizations, which meant that > if the optimizations managed to eliminate all the discard operations, > the flag would still be set, resulting (at least in theory) in slower > performance. > > This patch consolidates all of the code that sets UsesKill for GLSL > programs into do_set_program_inouts(), which already is doing a > similar job for UsesDFdy, and which occurs after i965's final round of > optimizations. > > Non-GLSL programs (ARB programs and the state tracker's glBitmap > program) are unaffected.
This series is: Reviewed-by: Eric Anholt <e...@anholt.net>
pgpcn42WT4mjT.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
