Oops. Thanks for tracking that down. Reviewed-by: Alex Smith <asm...@feralinteractive.com>
On 2 June 2018 at 13:31, Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> wrote: > Otherwise on pre-GFX9, if the constant layout allows both TESS_EVAL and > GEOMETRY shaders, but the PIPELINE has only GEOMETRY, it would return the > GEOMETRY shader for the TESS_EVAL shader. > > This would cause the flush_constants code to emit the GEOMETRY constants > to the TESS_EVAL registers and then conclude that it did not need to set > the GEOMETRY shader registers. > > Fixes: dfff9fb6f8d "radv: Handle GFX9 merged shaders in > radv_flush_constants()" > CC: 18.1 <mesa-sta...@lists.freedesktop.org> > CC: Alex Smith <asm...@feralinteractive.com> > CC: Samuel Pitoiset <samuel.pitoi...@gmail.com> > --- > src/amd/vulkan/radv_pipeline.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_ > pipeline.c > index ff647ed9af3..375f7c357d3 100644 > --- a/src/amd/vulkan/radv_pipeline.c > +++ b/src/amd/vulkan/radv_pipeline.c > @@ -1594,6 +1594,8 @@ radv_get_shader(struct radv_pipeline *pipeline, > if (pipeline->shaders[MESA_SHADER_GEOMETRY]) > return pipeline->shaders[MESA_SHADER_GEOMETRY]; > } else if (stage == MESA_SHADER_TESS_EVAL) { > + if (!radv_pipeline_has_tess(pipeline)) > + return NULL; > if (pipeline->shaders[MESA_SHADER_TESS_EVAL]) > return pipeline->shaders[MESA_SHADER_TESS_EVAL]; > if (pipeline->shaders[MESA_SHADER_GEOMETRY]) > -- > 2.17.0 > >
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev