From f3fad24b622d2fa51508e9a4066d393ef3260ddc Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Tue, 9 Jul 2013 13:30:15 -0600 Subject: [PATCH] tgsi: use X component of the second operand in exec_scalar_binary() The code happened to work in the past since the (scalar) src args effectively always have a swizzle of .xxxx, .yyyy, .zzzz, or .wwww so whether you grab the X or Y component doesn't really matter. Just fixing the code to make it look right. Reviewed-by: Roland Scheidegger --- src/gallium/auxiliary/tgsi/tgsi_exec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_exec.c b/src/gallium/auxiliary/tgsi/tgsi_exec.c index 4482c6bd12a..e89fb777afc 100644 --- a/src/gallium/auxiliary/tgsi/tgsi_exec.c +++ b/src/gallium/auxiliary/tgsi/tgsi_exec.c @@ -2521,7 +2521,7 @@ exec_scalar_binary(struct tgsi_exec_machine *mach, union tgsi_exec_channel dst; fetch_source(mach, &src[0], &inst->Src[0], TGSI_CHAN_X, src_datatype); - fetch_source(mach, &src[1], &inst->Src[1], TGSI_CHAN_Y, src_datatype); + fetch_source(mach, &src[1], &inst->Src[1], TGSI_CHAN_X, src_datatype); op(&dst, &src[0], &src[1]); for (chan = 0; chan < TGSI_NUM_CHANNELS; chan++) { if (inst->Dst[0].Register.WriteMask & (1 << chan)) { -- 2.30.2