On Thu, Jun 5, 2014 at 11:05 AM, Abdiel Janulgue
<abdiel.janul...@linux.intel.com> wrote:
> The negation source modifier on src registers has changed meaning in 
> Broadwell when
> used with logical operations. Don't copy propagate when negate src modifier 
> is set
> and when the destination instruction is a logical op.
>
> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com>
> ---
>  src/mesa/drivers/dri/i965/brw_vec4.h               |  4 +-
>  .../drivers/dri/i965/brw_vec4_copy_propagation.cpp | 68 
> +++++++++++++++-------
>  2 files changed, 49 insertions(+), 23 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4.h 
> b/src/mesa/drivers/dri/i965/brw_vec4.h
> index fd58b3c..51da46c 100644
> --- a/src/mesa/drivers/dri/i965/brw_vec4.h
> +++ b/src/mesa/drivers/dri/i965/brw_vec4.h
> @@ -228,6 +228,8 @@ writemask(dst_reg reg, unsigned mask)
>     return reg;
>  }
>
> +struct copy_entry;

I don't think this (specifically the new opcode field it was added
for) is used until the next patch?
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to