[Mesa-dev] [PATCH 08/18] i965/cs: Mark g0 as used by CS_OPCODE_CS_TERMINATE

Kenneth Graunke kenneth at whitecape.org
Mon Mar 16 13:44:24 PDT 2015


On Saturday, March 14, 2015 09:54:18 PM Jordan Justen wrote:
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> ---
>  src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp b/src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp
> index 72c490b..1f30a19 100644
> --- a/src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp
> @@ -405,6 +405,10 @@ fs_visitor::setup_payload_interference(struct ra_graph *g,
>           }
>           break;
>  
> +      case CS_OPCODE_CS_TERMINATE:
> +         payload_last_use_ip[0] = use_ip;
> +         break;
> +
>        default:
>           if (inst->eot) {
>              /* We could omit this for the !inst->header_present case, except
> 

I was going to ask why we didn't just fall through to the inst->eot case
below, but...it looks like the VS, GS, and FS all want g0/g1 reserved,
while compute only needs g0.  So this looks like a sensible solution.

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150316/0928888b/attachment.sig>


More information about the mesa-dev mailing list