Mesa (master): i965: Fix symbolic size of next_offset[] array.

Kenneth Graunke kwg at kemper.freedesktop.org
Fri Mar 24 19:22:12 UTC 2017


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Tue Feb 28 17:04:00 2017 -0800

i965: Fix symbolic size of next_offset[] array.

It's indexed by buffer, not stream.  BRW_MAX_SOL_BUFFERS and
MAX_VERTEX_STREAMS happen to both be 4, so there's no actual bug.

Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/mesa/drivers/dri/i965/gen7_sol_state.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/gen7_sol_state.c b/src/mesa/drivers/dri/i965/gen7_sol_state.c
index a4f46ea486..9964833c14 100644
--- a/src/mesa/drivers/dri/i965/gen7_sol_state.c
+++ b/src/mesa/drivers/dri/i965/gen7_sol_state.c
@@ -106,7 +106,7 @@ gen7_upload_3dstate_so_decl_list(struct brw_context *brw,
       xfb_obj->program->sh.LinkedTransformFeedback;
    uint16_t so_decl[MAX_VERTEX_STREAMS][128];
    int buffer_mask[MAX_VERTEX_STREAMS] = {0, 0, 0, 0};
-   int next_offset[MAX_VERTEX_STREAMS] = {0, 0, 0, 0};
+   int next_offset[BRW_MAX_SOL_BUFFERS] = {0, 0, 0, 0};
    int decls[MAX_VERTEX_STREAMS] = {0, 0, 0, 0};
    int max_decls = 0;
    STATIC_ASSERT(ARRAY_SIZE(so_decl[0]) >= MAX_PROGRAM_OUTPUTS);




More information about the mesa-commit mailing list