From: Matt Turner Date: Mon, 11 Aug 2014 18:12:43 +0000 (-0700) Subject: i965: Return NONE from brw_swap_cmod on unknown input. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e005c1148db04cb706a65392c2b0beda19efa0b1;p=mesa.git i965: Return NONE from brw_swap_cmod on unknown input. Comparing ~0u with a packed enum (i.e., 1 byte) always evaluates to false. Shouldn't gcc warn about this? Reported-by: Connor Abbott Reviewed-by: Kenneth Graunke --- diff --git a/src/mesa/drivers/dri/i965/brw_eu.c b/src/mesa/drivers/dri/i965/brw_eu.c index b4d5d882883..990597286d8 100644 --- a/src/mesa/drivers/dri/i965/brw_eu.c +++ b/src/mesa/drivers/dri/i965/brw_eu.c @@ -84,7 +84,7 @@ brw_swap_cmod(uint32_t cmod) case BRW_CONDITIONAL_LE: return BRW_CONDITIONAL_GE; default: - return ~0; + return BRW_CONDITIONAL_NONE; } } diff --git a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp index 071620279b2..498e0782f69 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp @@ -440,7 +440,7 @@ try_constant_propagate(struct brw_context *brw, fs_inst *inst, enum brw_conditional_mod new_cmod; new_cmod = brw_swap_cmod(inst->conditional_mod); - if (new_cmod != ~0u) { + if (new_cmod != BRW_CONDITIONAL_NONE) { /* Fit this constant in by swapping the operands and * flipping the test */ diff --git a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp index 390448ad801..28f162df5ba 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp @@ -165,7 +165,7 @@ try_constant_propagate(struct brw_context *brw, vec4_instruction *inst, enum brw_conditional_mod new_cmod; new_cmod = brw_swap_cmod(inst->conditional_mod); - if (new_cmod != ~0u) { + if (new_cmod != BRW_CONDITIONAL_NONE) { /* Fit this constant in by swapping the operands and * flipping the test. */