[Mesa-dev] [PATCH] r600: fix copy/paste bug for sampleMaskIn workaround
Eric Engestrom
eric.engestrom at intel.com
Mon Jun 18 10:43:12 UTC 2018
On Friday, 2018-06-15 23:38:58 +0200, sroland at vmware.com wrote:
> From: Roland Scheidegger <sroland at vmware.com>
>
> The sampleMaskIn workaround (b936f4d1ca0d2ab1e828ff6a6e617f12469687fa)
> tries to figure out if the shader is running at per-sample frequency, but
> there's a typo bug so it will only recognize per-sample linar inputs,
> not per-sample perspective ones.
>
> Spotted by Eric Engestrom <eric.engestrom at intel.com>
Fixes: b936f4d1ca0d2ab1e828a "r600: partly fix sampleMaskIn value"
(so that it goes on the stable branch as well)
> ---
> src/gallium/drivers/r600/r600_shader.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c
> index c9f2fa6485..c466a48262 100644
> --- a/src/gallium/drivers/r600/r600_shader.c
> +++ b/src/gallium/drivers/r600/r600_shader.c
> @@ -1247,7 +1247,7 @@ static int allocate_system_value_inputs(struct r600_shader_ctx *ctx, int gpr_off
> tgsi_parse_free(&parse);
>
> if (ctx->info.reads_samplemask &&
> - (ctx->info.uses_linear_sample || ctx->info.uses_linear_sample)) {
> + (ctx->info.uses_linear_sample || ctx->info.uses_persp_sample)) {
> inputs[1].enabled = true;
> }
>
> --
> 2.12.3
>
More information about the mesa-dev
mailing list