Re: [Mesa-dev] [PATCH] radv: enable VK_AMD_gpu_shader_half_float

2019-04-09 Thread Bas Nieuwenhuizen
a-b now. On Mon, Apr 8, 2019 at 9:31 AM Samuel Pitoiset wrote: > > > On 3/21/19 11:09 AM, Bas Nieuwenhuizen wrote: > > Honestly the zero tests is worrying me. This is a pretty big extension > > and I have questions like: > > > > to 16-bit loads + 16-bit ALU actually work together or have we been

Re: [Mesa-dev] [PATCH] radv: enable VK_AMD_gpu_shader_half_float

2019-04-08 Thread Samuel Pitoiset
On 3/21/19 11:09 AM, Bas Nieuwenhuizen wrote: Honestly the zero tests is worrying me. This is a pretty big extension and I have questions like: to 16-bit loads + 16-bit ALU actually work together or have we been silently relying on the fact there is always a ZExt cast after and that did not

Re: [Mesa-dev] [PATCH] radv: enable VK_AMD_gpu_shader_half_float

2019-03-21 Thread Samuel Pitoiset
On 3/21/19 11:09 AM, Bas Nieuwenhuizen wrote: Honestly the zero tests is worrying me. This is a pretty big extension and I have questions like: to 16-bit loads + 16-bit ALU actually work together or have we been silently relying on the fact there is always a ZExt cast after and that did not

Re: [Mesa-dev] [PATCH] radv: enable VK_AMD_gpu_shader_half_float

2019-03-21 Thread Bas Nieuwenhuizen
Honestly the zero tests is worrying me. This is a pretty big extension and I have questions like: to 16-bit loads + 16-bit ALU actually work together or have we been silently relying on the fact there is always a ZExt cast after and that did not care about input size? On Thu, Mar 21, 2019 at

[Mesa-dev] [PATCH] radv: enable VK_AMD_gpu_shader_half_float

2019-03-21 Thread Samuel Pitoiset
Should be safe to enable as all instructions seem to support 16-bit. Unfortunately, there is no CTS test. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_extensions.py | 1 + 1 file changed, 1 insertion(+) diff --git a/src/amd/vulkan/radv_extensions.py