Re: [Mesa-dev] [PATCH 17/22] i965/vec4: fix register_coalesce() for partial writes

2017-01-16 Thread Samuel Iglesias Gonsálvez
On Mon, 2017-01-16 at 09:01 +0100, Samuel Iglesias Gonsálvez wrote: > On Fri, 2017-01-13 at 15:46 -0800, Matt Turner wrote: > > On Thu, Jan 5, 2017 at 5:07 AM, Samuel Iglesias Gonsálvez > > wrote: > > > From: "Juan A. Suarez Romero" > > > > > > When lowering double_to_single() we added a final m

Re: [Mesa-dev] [PATCH 17/22] i965/vec4: fix register_coalesce() for partial writes

2017-01-16 Thread Samuel Iglesias Gonsálvez
On Fri, 2017-01-13 at 15:46 -0800, Matt Turner wrote: > On Thu, Jan 5, 2017 at 5:07 AM, Samuel Iglesias Gonsálvez > wrote: > > From: "Juan A. Suarez Romero" > > > > When lowering double_to_single() we added a final mov() that puts > > 32-bit > > I can't confirm that this patch is necessary in t

Re: [Mesa-dev] [PATCH 17/22] i965/vec4: fix register_coalesce() for partial writes

2017-01-13 Thread Matt Turner
On Thu, Jan 5, 2017 at 5:07 AM, Samuel Iglesias Gonsálvez wrote: > From: "Juan A. Suarez Romero" > > When lowering double_to_single() we added a final mov() that puts 32-bit I can't confirm that this patch is necessary in the current i965-fp64-gen7-ivb-scalar-vec4-rc2 branch. It passes Jenkins w

[Mesa-dev] [PATCH 17/22] i965/vec4: fix register_coalesce() for partial writes

2017-01-05 Thread Samuel Iglesias Gonsálvez
From: "Juan A. Suarez Romero" When lowering double_to_single() we added a final mov() that puts 32-bit values from one register in the second half of destination. --- src/mesa/drivers/dri/i965/brw_vec4.cpp | 5 + src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 3 ++