[Mesa-dev] [PATCH] r600g/cayman: fix texture gather tests
Glenn Kennard
glenn.kennard at gmail.com
Mon Nov 17 17:46:57 PST 2014
On Tue, 18 Nov 2014 01:57:13 +0100, Dave Airlie <airlied at gmail.com> wrote:
> From: Dave Airlie <airlied at redhat.com>
>
> It appears on cayman the TG4 outputs were reordered.
>
> This fixes a lot of piglit tests.
>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
> ---
> src/gallium/drivers/r600/r600_shader.c | 15 +++++++++++----
> 1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/drivers/r600/r600_shader.c
> b/src/gallium/drivers/r600/r600_shader.c
> index 4c6ae45..709fcd7 100644
> --- a/src/gallium/drivers/r600/r600_shader.c
> +++ b/src/gallium/drivers/r600/r600_shader.c
> @@ -5763,11 +5763,18 @@ static int tgsi_tex(struct r600_shader_ctx *ctx)
> int8_t texture_component_select = ctx->literals[4 *
> inst->Src[1].Register.Index + inst->Src[1].Register.SwizzleX];
> tex.inst_mod = texture_component_select;
> + if (ctx->bc->chip_class == CAYMAN) {
> /* GATHER4 result order is different from TGSI TG4 */
> - tex.dst_sel_x = (inst->Dst[0].Register.WriteMask & 2) ? 1 : 7;
> - tex.dst_sel_y = (inst->Dst[0].Register.WriteMask & 4) ? 2 : 7;
> - tex.dst_sel_z = (inst->Dst[0].Register.WriteMask & 1) ? 0 : 7;
> - tex.dst_sel_w = (inst->Dst[0].Register.WriteMask & 8) ? 3 : 7;
> + tex.dst_sel_x = (inst->Dst[0].Register.WriteMask & 2) ? 0 : 7;
> + tex.dst_sel_y = (inst->Dst[0].Register.WriteMask & 4) ? 1 : 7;
> + tex.dst_sel_z = (inst->Dst[0].Register.WriteMask & 1) ? 2 : 7;
> + tex.dst_sel_w = (inst->Dst[0].Register.WriteMask & 8) ? 3 : 7;
> + } else {
> + tex.dst_sel_x = (inst->Dst[0].Register.WriteMask & 2) ? 1 : 7;
> + tex.dst_sel_y = (inst->Dst[0].Register.WriteMask & 4) ? 2 : 7;
> + tex.dst_sel_z = (inst->Dst[0].Register.WriteMask & 1) ? 0 : 7;
> + tex.dst_sel_w = (inst->Dst[0].Register.WriteMask & 8) ? 3 : 7;
> + }
> }
> else if (inst->Instruction.Opcode == TGSI_OPCODE_LODQ) {
> tex.dst_sel_x = (inst->Dst[0].Register.WriteMask & 2) ? 1 : 7;
Gotta permute those tex op bit encodings between hardware generations or
they go stale...
Reviewed-by: Glenn Kennard <glenn.kennard at gmail.com>
More information about the mesa-dev
mailing list