anv: emit scissors when the pipeline changes
authorIván Briano <ivan.briano@intel.com>
Tue, 9 Aug 2022 18:41:31 +0000 (11:41 -0700)
committerMarge Bot <emma+marge@anholt.net>
Tue, 9 Aug 2022 19:58:14 +0000 (19:58 +0000)
commitfbd41337354326354739042368989535d79f1640
treeb378534969e646c0747c091a58a6e91f843734e0
parentdd7cae2a9960db528f65036e85412c3f51afd6a0
anv: emit scissors when the pipeline changes

With the switch to common dynamic state tracking, something got lost
that made the scissors not always be emitted when they are not dynamic
and the pipeline is marked dirty.

Since both viewport and scissors make use of each other to calculate
their values, just stick the scissor emit in the same if block as
viewport for now.
I'd rather have them decoupled, and at least the Vulkan CTS didn't
complain when I tried it, but I don't know what other effects that
may have, especially when it comes to the guardband.

Fixes a bunch of tests under
dEQP-VK.pipeline.*.multisample.misc.*

Fixes: 7d25c04236b ("anv: Switch to using common dynamic state tracking")

Reviewed-by: Jason Ekstrand <jason.ekstrand@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17964>
src/intel/vulkan/genX_cmd_buffer.c