[Mesa-stable] [PATCH] radv: fix centroid interpolation

Mark Janes mark.a.janes at intel.com
Mon May 21 16:30:25 UTC 2018


Samuel Pitoiset <samuel.pitoiset at gmail.com> writes:

> It's legal to set the centroid and sample interpolation modes
> when MSAA disabled. So, we have to initialize the centroid
> inputs because the hardware doesn't.
>
> This fixes rendering issues with DXVK and The Witness, World of
> Warcraft, Trackmania and probably more games.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106315
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102390
> CC: 18.0 18.1 <mesa-stable at lists.freedesktop.org>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/amd/vulkan/radv_nir_to_llvm.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/src/amd/vulkan/radv_nir_to_llvm.c b/src/amd/vulkan/radv_nir_to_llvm.c
> index 9b9619b877e..2d91ded7fe5 100644
> --- a/src/amd/vulkan/radv_nir_to_llvm.c
> +++ b/src/amd/vulkan/radv_nir_to_llvm.c
> @@ -2075,9 +2075,6 @@ static void
>  prepare_interp_optimize(struct radv_shader_context *ctx,
>                          struct nir_shader *nir)
>  {
> -	if (!ctx->options->key.fs.multisample)
> -		return;
> -

Mesa 18.0 is CC'd on this patch, but the hunk is not in that branch.

>  	bool uses_center = false;
>  	bool uses_centroid = false;
>  	nir_foreach_variable(variable, &nir->inputs) {
> -- 
> 2.17.0
>
> _______________________________________________
> mesa-stable mailing list
> mesa-stable at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-stable


More information about the mesa-stable mailing list