[Mesa-dev] [PATCH v2 056/103] i965/disasm: fix subreg for dst in Align16 mode

Iago Toral Quiroga itoral at igalia.com
Tue Oct 11 09:02:00 UTC 2016


There is a single bit for this, so it is a binary 0 or 1 meaning
offset 0B or 16B respectively.

v2:
  - Since brw_inst_dst_da16_subreg_nr() is known to be 1, remove it
    from the expression (Curro)

Reviewed-by: Francisco Jerez <currojerez at riseup.net>
---
 src/mesa/drivers/dri/i965/brw_disasm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c b/src/mesa/drivers/dri/i965/brw_disasm.c
index 0c43217..e439ec4 100644
--- a/src/mesa/drivers/dri/i965/brw_disasm.c
+++ b/src/mesa/drivers/dri/i965/brw_disasm.c
@@ -772,7 +772,7 @@ dest(FILE *file, const struct gen_device_info *devinfo, brw_inst *inst)
          if (err == -1)
             return 0;
          if (brw_inst_dst_da16_subreg_nr(devinfo, inst))
-            format(file, ".%"PRIu64, brw_inst_dst_da16_subreg_nr(devinfo, inst) /
+            format(file, ".%u", 16 /
                    reg_type_size[brw_inst_dst_reg_type(devinfo, inst)]);
          string(file, "<1>");
          err |= control(file, "writemask", writemask,
-- 
2.7.4



More information about the mesa-dev mailing list