Re: [Mesa-dev] [PATCH] radv/ac: canonicalize the output for 32-bit float min/max.

2017-05-02 Thread Dave Airlie
On 17 March 2017 at 18:24, Bas Nieuwenhuizen wrote: > Where do you see that they are required to flush denormals to 0? I > can't find it in the GLSL extensions for SPIR-V, and the vulkan spec > has the following note: > > "Any denormalized value input into a shader or potentially generated > by an

Re: [Mesa-dev] [PATCH] radv/ac: canonicalize the output for 32-bit float min/max.

2017-03-17 Thread Bas Nieuwenhuizen
Where do you see that they are required to flush denormals to 0? I can't find it in the GLSL extensions for SPIR-V, and the vulkan spec has the following note: "Any denormalized value input into a shader or potentially generated by any instruction in a shader may be flushed to 0." Which very much

Re: [Mesa-dev] [PATCH] radv/ac: canonicalize the output for 32-bit float min/max.

2017-03-16 Thread Dave Airlie
Doh missent two patches, will rebase and resend. Dave. > > This fixes: > dEQP-VK.glsl.builtin.precision.min.* > dEQP-VK.glsl.builtin.precision.max.* > dEQP-VK.glsl.builtin.precision.clamp.* > > As weren't flushing the denorms as SPIR-V required. > > Signed-off-by: Dave Airlie > --- > src/amd/com

[Mesa-dev] [PATCH] radv/ac: canonicalize the output for 32-bit float min/max.

2017-03-16 Thread Dave Airlie
From: Dave Airlie This fixes: dEQP-VK.glsl.builtin.precision.min.* dEQP-VK.glsl.builtin.precision.max.* dEQP-VK.glsl.builtin.precision.clamp.* As weren't flushing the denorms as SPIR-V required. Signed-off-by: Dave Airlie --- src/amd/common/ac_nir_to_llvm.c | 14 -- 1 file changed