On Thu, Jan 5, 2017 at 5:07 AM, Samuel Iglesias Gonsálvez <sigles...@igalia.com> wrote: > From: "Juan A. Suarez Romero" <jasua...@igalia.com> > > When spliting double_to_single() in Ivybridge/Valleyview, the second > part should use a temporal register, and then move the values to the > second half of the original destiny, so we get all the results in the
Typo: destination > same register. Please change double_to_single() to VEC4_OPCODE_FROM_DOUBLE (or just FROM_DOUBLE) throughout. > --- > src/mesa/drivers/dri/i965/brw_vec4.cpp | 17 +++++++++++++---- > src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 1 + > 2 files changed, 14 insertions(+), 4 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp > b/src/mesa/drivers/dri/i965/brw_vec4.cpp > index f533207..afabc22 100644 > --- a/src/mesa/drivers/dri/i965/brw_vec4.cpp > +++ b/src/mesa/drivers/dri/i965/brw_vec4.cpp > @@ -2199,9 +2199,15 @@ vec4_visitor::lower_simd_width() > linst->group = channel_offset; > linst->size_written = size_written; > > + /* When spliting double_to_single() in Ivybridge, the second part Typo: splitting s/in/on/ _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev