On Monday, August 25, 2014 06:41:54 PM Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_disasm.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c
> b/src/mesa/drivers/dri/i965/brw_disasm.c
> index 424b214..65c6f23 100644
> --- a/src/mesa/drivers/dri/i965/brw_disasm.c
> +++ b/src/mesa/drivers/dri/i965/brw_disasm.c
> @@ -137,8 +137,8 @@ is_logic_instruction(unsigned opcode)
>
> const char *const conditional_modifier[16] = {
> [BRW_CONDITIONAL_NONE] = "",
> - [BRW_CONDITIONAL_Z] = ".e",
> - [BRW_CONDITIONAL_NZ] = ".ne",
> + [BRW_CONDITIONAL_Z] = ".z",
> + [BRW_CONDITIONAL_NZ] = ".nz",
> [BRW_CONDITIONAL_G] = ".g",
> [BRW_CONDITIONAL_GE] = ".ge",
> [BRW_CONDITIONAL_L] = ".l",Both are used in the docs, so it's kind of a toss-up. I'm fine with this, though, and the rest of the series looks great. Rather embarassing that we were disassembling PopCount as JumpCount :( All 4 are: Reviewed-by: Kenneth Graunke <[email protected]>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
