tgsi: use X component of the second operand in exec_scalar_binary()
authorBrian Paul <brianp@vmware.com>
Tue, 9 Jul 2013 19:30:15 +0000 (13:30 -0600)
committerBrian Paul <brianp@vmware.com>
Fri, 12 Jul 2013 14:32:51 +0000 (08:32 -0600)
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 <sroland@vmware.com>
src/gallium/auxiliary/tgsi/tgsi_exec.c

index 4482c6bd12a6490eb193ce24b21f65a4b4ff3530..e89fb777afcb876ecc3f5dd834ea02b073107c99 100644 (file)
@@ -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)) {