[Mesa-dev] [PATCH v2 01/20] i965/disasm: also print nibctrl in IVB for execsize=8

Francisco Jerez currojerez at riseup.net
Tue Jan 17 20:33:50 UTC 2017


Samuel Iglesias Gonsálvez <siglesias at igalia.com> writes:

> From: Iago Toral Quiroga <itoral at igalia.com>
>
> 4-wide DF operations where NibCtrl applies require and execsize of 8
> in IvyBridge/BayTrail.
>
> v2:
> - Refactor NibCtrl printing (Matt)
>
> Reviewed-by: Matt Turner <mattst88 at gmail.com>

Reviewed-by: Francisco Jerez <currojerez at riseup.net>

> ---
>  src/mesa/drivers/dri/i965/brw_disasm.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c b/src/mesa/drivers/dri/i965/brw_disasm.c
> index 167067a4b7d..b04325564a6 100644
> --- a/src/mesa/drivers/dri/i965/brw_disasm.c
> +++ b/src/mesa/drivers/dri/i965/brw_disasm.c
> @@ -1189,10 +1189,10 @@ qtr_ctrl(FILE *file, const struct gen_device_info *devinfo, brw_inst *inst)
>  {
>     int qtr_ctl = brw_inst_qtr_control(devinfo, inst);
>     int exec_size = 1 << brw_inst_exec_size(devinfo, inst);
> +   const unsigned nib_ctl = devinfo->gen < 7 ? 0 :
> +                            brw_inst_nib_control(devinfo, inst);
>  
> -   if (exec_size < 8) {
> -      const unsigned nib_ctl = devinfo->gen < 7 ? 0 :
> -                               brw_inst_nib_control(devinfo, inst);
> +   if (exec_size < 8 || nib_ctl) {
>        format(file, " %dN", qtr_ctl * 2 + nib_ctl + 1);
>     } else if (exec_size == 8) {
>        switch (qtr_ctl) {
> -- 
> 2.11.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170117/532c14d7/attachment-0001.sig>


More information about the mesa-dev mailing list