Re: [Mesa-dev] [PATCH v2] nir/lower_double_ops: lower trunc()

2016-04-24 Thread Iago Toral
On Fri, 2016-04-22 at 15:06 -0700, Jason Ekstrand wrote: > Reviewed-by: Jason Ekstrand > > > Did the v2 fix anything other than newly added tests? No, only the new tests that targeted the cases where the unbiased exponent was between 0 and 52. Iago > --Jason > > > On Thu, Apr 21, 2016 at 10

Re: [Mesa-dev] [PATCH v2] nir/lower_double_ops: lower trunc()

2016-04-22 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand Did the v2 fix anything other than newly added tests? --Jason On Thu, Apr 21, 2016 at 10:13 PM, Samuel Iglesias Gonsálvez < sigles...@igalia.com> wrote: > From: Iago Toral Quiroga > > At least i965 hardware does not have native support for truncating doubles. > > v2

[Mesa-dev] [PATCH v2] nir/lower_double_ops: lower trunc()

2016-04-21 Thread Samuel Iglesias Gonsálvez
From: Iago Toral Quiroga At least i965 hardware does not have native support for truncating doubles. v2: - Simplified the implementation significantly. - Fixed the else branch, that was not doing what we wanted. --- src/compiler/nir/nir.h | 1 + src/compiler/nir/nir_lower_