i965/fs: Set LastRT on the final FB write on Broadwell.
authorKenneth Graunke <kenneth@whitecape.org>
Mon, 21 Jul 2014 23:17:46 +0000 (16:17 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 23 Jul 2014 22:44:37 +0000 (15:44 -0700)
commitd4d886a0bc86eb2045b0327fe147eef3c400ffba
tree38f8ee13ec73af9824a429006d747e0d6c8bfd6a
parent36a4a6bbdca0c30e16d56e6b406ea7c94831048f
i965/fs: Set LastRT on the final FB write on Broadwell.

In Piglit's EXT_framebuffer_multisample/alpha-to-coverage-dual-src-blend
test, key->nr_color_regions == 2, but the dual source blend FB write has
ir->target set to 0.  So we failed to set "Last Render Target Select" on
any FB write message.

We only emit one FB write per render target, so my comment about setting
LastRT on every FB write directed at the last color region is a bit...
misinformed.  According to the documentation, depth buffer writes and
scoreboard updates happen on the FB write with LastRT set, so I believe
we want to set it only once.

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Cc: "10.2" <mesa-stable@lists.freedesktop.org>
src/mesa/drivers/dri/i965/gen8_fs_generator.cpp