Mesa (master): i965: Initialize "separate" flag in VUE maps.

Kenneth Graunke kwg at kemper.freedesktop.org
Wed Dec 7 06:26:41 UTC 2016


Module: Mesa
Branch: master
Commit: 09ffc5c84f8067e3b8d0ee89a1771e0a38595036
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=09ffc5c84f8067e3b8d0ee89a1771e0a38595036

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Tue Dec  6 02:43:07 2016 -0800

i965: Initialize "separate" flag in VUE maps.

This was uninitialized, which resulted in weird looking printouts where
it appeared that the TCS output and TES input patch URB entries differed
in SSO/non-SSO layout.  There is no "separable" layout for both, as
they're tied together.

It should have no other actual effect.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/mesa/drivers/dri/i965/brw_vue_map.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/mesa/drivers/dri/i965/brw_vue_map.c b/src/mesa/drivers/dri/i965/brw_vue_map.c
index 4d509d3..982a31f 100644
--- a/src/mesa/drivers/dri/i965/brw_vue_map.c
+++ b/src/mesa/drivers/dri/i965/brw_vue_map.c
@@ -205,6 +205,9 @@ brw_compute_tess_vue_map(struct brw_vue_map *vue_map,
    /* I don't think anything actually uses this... */
    vue_map->slots_valid = vertex_slots;
 
+   /* separate isn't really meaningful, but make sure it's initialized */
+   vue_map->separate = false;
+
    vertex_slots &= ~(VARYING_BIT_TESS_LEVEL_OUTER |
                      VARYING_BIT_TESS_LEVEL_INNER);
 




More information about the mesa-commit mailing list