[Mesa-dev] [PATCH 4/4] i965/vec4: Invalidate live intervals in opt_cse, not _local.

Kenneth Graunke kenneth at whitecape.org
Sat Jul 12 00:19:11 PDT 2014


On Friday, July 11, 2014 08:50:01 PM Matt Turner wrote:
> ---
>  src/mesa/drivers/dri/i965/brw_vec4_cse.cpp | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_cse.cpp 
b/src/mesa/drivers/dri/i965/brw_vec4_cse.cpp
> index 7bb016d..83c7eb8 100644
> --- a/src/mesa/drivers/dri/i965/brw_vec4_cse.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_vec4_cse.cpp
> @@ -244,9 +244,6 @@ vec4_visitor::opt_cse_local(bblock_t *block)
>  
>     ralloc_free(cse_ctx);
>  
> -   if (progress)
> -      invalidate_live_intervals();
> -
>     return progress;
>  }
>  
> @@ -265,5 +262,8 @@ vec4_visitor::opt_cse()
>        progress = opt_cse_local(block) || progress;
>     }
>  
> +   if (progress)
> +      invalidate_live_intervals();
> +
>     return progress;
>  }
> 

Series is:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

Not sure why I wrote such silly code in the first place - thanks for cleaning 
it up.
-------------- 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/20140712/cf69b851/attachment.sig>


More information about the mesa-dev mailing list