vbo: Don't set node->min_index = max_index = indices_offset when merging
authorKenneth Graunke <kenneth@whitecape.org>
Thu, 14 Jan 2021 23:42:01 +0000 (15:42 -0800)
committerMarge Bot <eric+marge@anholt.net>
Wed, 20 Jan 2021 00:29:04 +0000 (00:29 +0000)
commit44bdd5225c48d266e42d93defbb7ca9ae74aa21b
treefbd97d01ff0aa147fa2862e2be3f5e1df39d95f8
parentdd72019d03f4df82cdce982d78f7983f32207929
vbo: Don't set node->min_index = max_index = indices_offset when merging

I'm can't see why this is necessary.  There are already new fields
(node->merged.{min,max}_index) for the new values in the merged case.
But in vbo_save_draw.c, in the !draw_using_merged_prim case, we would
try and use the original node...with the now destroyed min/max index.

Fixes some assert failures when running with swtnl and forcing the
non-merged path (though it takes the merged path by default).

Fixes: 4c751ad67aa ("vbo/dlist: use a shared index buffer")
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8522>
src/mesa/vbo/vbo_save_api.c