Re: [Mesa-dev] [PATCH 4/5] i965: Don't try copy propagation if constant propagation succeeded.

2016-03-03 Thread Matt Turner
On Mon, Feb 29, 2016 at 4:43 PM, Francisco Jerez wrote: > Matt Turner writes: > >>> i965: Don't try copy propagation if constant propagation succeeded. >>> >>> It cannot get any better. >> >> That's not obvious to me. How do we know that? > > Because

Re: [Mesa-dev] [PATCH 4/5] i965: Don't try copy propagation if constant propagation succeeded.

2016-02-29 Thread Francisco Jerez
Matt Turner writes: >> i965: Don't try copy propagation if constant propagation succeeded. >> >> It cannot get any better. > > That's not obvious to me. How do we know that? Because it's impossible to copy-propagate anything into a source that's already an immediate. ;)

Re: [Mesa-dev] [PATCH 4/5] i965: Don't try copy propagation if constant propagation succeeded.

2016-02-29 Thread Matt Turner
> i965: Don't try copy propagation if constant propagation succeeded. > > It cannot get any better. That's not obvious to me. How do we know that? ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH 4/5] i965: Don't try copy propagation if constant propagation succeeded.

2016-02-29 Thread Iago Toral
On Fri, 2016-02-26 at 22:02 -0800, Francisco Jerez wrote: > It cannot get any better. > --- > src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 2 +- > src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git

[Mesa-dev] [PATCH 4/5] i965: Don't try copy propagation if constant propagation succeeded.

2016-02-26 Thread Francisco Jerez
It cannot get any better. --- src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 2 +- src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp