Re: [Mesa-dev] [PATCH v2] nir/spirv/glsl450: rewrite atan2 to deal with infinities

2017-01-24 Thread Francisco Jerez
Francisco Jerez writes: > "Juan A. Suarez Romero" writes: > >> On Sun, 2017-01-22 at 00:20 -0800, Francisco Jerez wrote: >>> "Juan A. Suarez Romero" writes: >>> >>> > Rewrite atan2(y,x) to cover (+/-)INF values. >>> > >>> >

Re: [Mesa-dev] [PATCH v2] nir/spirv/glsl450: rewrite atan2 to deal with infinities

2017-01-23 Thread Francisco Jerez
"Juan A. Suarez Romero" writes: > On Sun, 2017-01-22 at 00:20 -0800, Francisco Jerez wrote: >> "Juan A. Suarez Romero" writes: >> >> > Rewrite atan2(y,x) to cover (+/-)INF values. >> > >> > This fixes several test cases in Vulkan CTS >> >

Re: [Mesa-dev] [PATCH v2] nir/spirv/glsl450: rewrite atan2 to deal with infinities

2017-01-23 Thread Juan A. Suarez Romero
On Sun, 2017-01-22 at 00:20 -0800, Francisco Jerez wrote: > "Juan A. Suarez Romero" writes: > > > Rewrite atan2(y,x) to cover (+/-)INF values. > > > > This fixes several test cases in Vulkan CTS > > (dEQP-VK.glsl.builtin.precision.atan2.*) > > > > v2: do not flush denorms

Re: [Mesa-dev] [PATCH v2] nir/spirv/glsl450: rewrite atan2 to deal with infinities

2017-01-22 Thread Francisco Jerez
"Juan A. Suarez Romero" writes: > Rewrite atan2(y,x) to cover (+/-)INF values. > > This fixes several test cases in Vulkan CTS > (dEQP-VK.glsl.builtin.precision.atan2.*) > > v2: do not flush denorms to 0 (jasuarez) > --- > src/compiler/spirv/vtn_glsl450.c | 48 >

[Mesa-dev] [PATCH v2] nir/spirv/glsl450: rewrite atan2 to deal with infinities

2017-01-18 Thread Juan A. Suarez Romero
Rewrite atan2(y,x) to cover (+/-)INF values. This fixes several test cases in Vulkan CTS (dEQP-VK.glsl.builtin.precision.atan2.*) v2: do not flush denorms to 0 (jasuarez) --- src/compiler/spirv/vtn_glsl450.c | 48 +++- 1 file changed, 42 insertions(+), 6