Mesa (master): gallivm: Fix wrong operator in lp_exec_default.

Jose Fonseca jrfonseca at kemper.freedesktop.org
Thu Apr 24 13:54:05 UTC 2014


Module: Mesa
Branch: master
Commit: 1527a545a4ebb825db02bba9c9e42a90c15326f6
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=1527a545a4ebb825db02bba9c9e42a90c15326f6

Author: José Fonseca <jfonseca at vmware.com>
Date:   Thu Apr 24 14:49:53 2014 +0100

gallivm: Fix wrong operator in lp_exec_default.

Courtesy of MSVC static code analyser.

Reviewed-by: Roland Scheidegger <sroland at vmware.com>

---

 src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c b/src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c
index 8791168..2b47fc2 100644
--- a/src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c
+++ b/src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c
@@ -725,7 +725,7 @@ static void lp_exec_default(struct lp_exec_mask *mask,
        * default (or could do switch analysis at switch start time instead).
        */
       unsigned opcode = bld_base->instructions[bld_base->pc - 1].Instruction.Opcode;
-      boolean ft_into = (opcode != TGSI_OPCODE_BRK ||
+      boolean ft_into = (opcode != TGSI_OPCODE_BRK &&
                          opcode != TGSI_OPCODE_SWITCH);
       /*
        * If it is not last statement and there was no fallthrough into it,




More information about the mesa-commit mailing list