intel/vec4/gs: reset nr_pull_param if DUAL_INSTANCED compile failed.
authorDave Airlie <airlied@gmail.com>
Thu, 3 Aug 2017 03:48:40 +0000 (13:48 +1000)
committerDave Airlie <airlied@gmail.com>
Thu, 3 Aug 2017 06:54:08 +0000 (16:54 +1000)
commit271fa3a684ef0eefe99087c13d1abb099784163f
tree1103f5b73ca482f2e0abefa44abf82a8d4344ce9
parentd5ba75f8881f0869dc16f71f7395514c0a35b6e2
intel/vec4/gs: reset nr_pull_param if DUAL_INSTANCED compile failed.

If dual object compile fails (as seems to happen with virgl a
fair bit, and does piglit even have any tests for it?), we end up
not restarting the pull params, so we call
vec4_visitor::move_uniform_array_access_to_pull_constant
a second time and it runs over the ends of the alloc.

Fixes: tests/spec/glsl-1.50/execution/geometry/max-input-components.shader_test
running inside virgl on ivybridge.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Cc: <mesa-stable@lists.freedesktop.org>
Signed-off-by: Dave Airlie <airlied@redhat.com>
src/intel/compiler/brw_vec4_gs_visitor.cpp