[Mesa-dev] [PATCH] i965/cs: Use exec all for CS terminate

[Francisco Jerez]

Jordan Justen <jordan.l.justen at intel.com> writes:

> This prevents an assertion from being hit with SIMD16:
>
> Assertion `inst->exec_size == dispatch_width() || force_writemask_all' failed.
>
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> Cc: Francisco Jerez <currojerez at riseup.net>
> ---
>  src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> index c41284b..588966b 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> @@ -1948,7 +1948,8 @@ fs_visitor::emit_cs_terminate()
>     bld.exec_all().MOV(payload, g0);
>  
>     /* Send a message to the thread spawner to terminate the thread. */
> -   fs_inst *inst = bld.emit(CS_OPCODE_CS_TERMINATE, reg_undef, payload);
> +   fs_inst *inst = bld.exec_all()
> +                      .emit(CS_OPCODE_CS_TERMINATE, reg_undef, payload);
>     inst->eot = true;
>  }
>  
> -- 
> 2.1.4

Reviewed-by: Francisco Jerez <currojerez at riseup.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150612/7fb58269/attachment.sig>