Re: [Mesa-dev] [PATCH v4 19/28] i965/vec4: fix SIMD-width lowering for VEC4_OPCODE_FROM_DOUBLE

2017-03-23 Thread Samuel Iglesias Gonsálvez
On Thu, 2017-03-23 at 12:14 -0700, Francisco Jerez wrote: > Samuel Iglesias Gonsálvez writes: > > > On Wed, 2017-03-22 at 15:55 -0700, Francisco Jerez wrote: > > > Samuel Iglesias Gonsálvez writes: > > > > > > > Now the VEC4_OPCODE_FROM_DOUBLE's destination data is written > > > > with > > > >

Re: [Mesa-dev] [PATCH v4 19/28] i965/vec4: fix SIMD-width lowering for VEC4_OPCODE_FROM_DOUBLE

2017-03-23 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > On Wed, 2017-03-22 at 15:55 -0700, Francisco Jerez wrote: >> Samuel Iglesias Gonsálvez writes: >> >> > Now the VEC4_OPCODE_FROM_DOUBLE's destination data is written with >> > stride 2. We need to take into account this when doing the split >> > so we don't ov

Re: [Mesa-dev] [PATCH v4 19/28] i965/vec4: fix SIMD-width lowering for VEC4_OPCODE_FROM_DOUBLE

2017-03-23 Thread Samuel Iglesias Gonsálvez
On Wed, 2017-03-22 at 15:55 -0700, Francisco Jerez wrote: > Samuel Iglesias Gonsálvez writes: > > > Now the VEC4_OPCODE_FROM_DOUBLE's destination data is written with > > stride 2. We need to take into account this when doing the split > > so we don't overwrite data. > > > > You should probably

Re: [Mesa-dev] [PATCH v4 19/28] i965/vec4: fix SIMD-width lowering for VEC4_OPCODE_FROM_DOUBLE

2017-03-22 Thread Francisco Jerez
Samuel Iglesias Gonsálvez writes: > Now the VEC4_OPCODE_FROM_DOUBLE's destination data is written with > stride 2. We need to take into account this when doing the split > so we don't overwrite data. > You should probably fix the destination type of your VEC4_OPCODE_FROM_DOUBLE instructions in P

[Mesa-dev] [PATCH v4 19/28] i965/vec4: fix SIMD-width lowering for VEC4_OPCODE_FROM_DOUBLE

2017-03-20 Thread Samuel Iglesias Gonsálvez
Now the VEC4_OPCODE_FROM_DOUBLE's destination data is written with stride 2. We need to take into account this when doing the split so we don't overwrite data. Signed-off-by: Samuel Iglesias Gonsálvez --- src/intel/compiler/brw_vec4.cpp | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-)