Re: [Mesa-dev] [PATCH v2] nir/spirv: fix chain access with different index bit sizes

2017-09-07 Thread Jason Ekstrand
I'd probably go with u2u32 so we don't get sign-extension. Other than that, Reviewed-by: Jason Ekstrand On Mon, Aug 28, 2017 at 11:42 PM, Samuel Iglesias Gonsálvez < sigles...@igalia.com> wrote: > Currently we support 32-bit indexes/offsets all over the driver, so we >

Re: [Mesa-dev] [PATCH v2] nir/spirv: fix chain access with different index bit sizes

2017-09-07 Thread Samuel Iglesias Gonsálvez
This patch is unreviewed. On Tue, 2017-08-29 at 08:42 +0200, Samuel Iglesias Gonsálvez wrote: > Currently we support 32-bit indexes/offsets all over the driver, so > we > convert them to that bit size. > > Fixes dEQP-VK.spirv_assembly.instruction.*.indexing.* > > Signed-off-by: Samuel Iglesias

[Mesa-dev] [PATCH v2] nir/spirv: fix chain access with different index bit sizes

2017-08-29 Thread Samuel Iglesias Gonsálvez
Currently we support 32-bit indexes/offsets all over the driver, so we convert them to that bit size. Fixes dEQP-VK.spirv_assembly.instruction.*.indexing.* Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/vtn_variables.c | 11 --- 1 file changed, 8