Re: [Mesa-dev] [PATCH] spirv: consider bitsize when handling OpSwitch cases

2017-12-20 Thread Jason Ekstrand
On December 20, 2017 09:53:13 Emil Velikov wrote: On 19 December 2017 at 18:05, Juan A. Suarez Romero wrote: When walking over all the cases in a OpSwitch, take in account the bitsize of the literals to avoid getting wrong cases. Even if

Re: [Mesa-dev] [PATCH] spirv: consider bitsize when handling OpSwitch cases

2017-12-20 Thread Emil Velikov
On 19 December 2017 at 18:05, Juan A. Suarez Romero wrote: > When walking over all the cases in a OpSwitch, take in account the bitsize > of the literals to avoid getting wrong cases. Even if there's no obvious tests that this fixes a stable and/or fixes tag will be a good

Re: [Mesa-dev] [PATCH] spirv: consider bitsize when handling OpSwitch cases

2017-12-19 Thread Jason Ekstrand
Oops, thanks. Rb On December 19, 2017 12:05:43 "Juan A. Suarez Romero" wrote: When walking over all the cases in a OpSwitch, take in account the bitsize of the literals to avoid getting wrong cases. --- src/compiler/spirv/vtn_cfg.c | 14 +++--- 1 file

Re: [Mesa-dev] [PATCH] spirv: consider bitsize when handling OpSwitch cases

2017-12-19 Thread Juan A. Suarez Romero
Sorry, sent twice the same patch. J.A. On Tue, 2017-12-19 at 19:05 +0100, Juan A. Suarez Romero wrote: > When walking over all the cases in a OpSwitch, take in account the bitsize > of the literals to avoid getting wrong cases. > --- > src/compiler/spirv/vtn_cfg.c | 14 +++--- >

[Mesa-dev] [PATCH] spirv: consider bitsize when handling OpSwitch cases

2017-12-19 Thread Juan A. Suarez Romero
When walking over all the cases in a OpSwitch, take in account the bitsize of the literals to avoid getting wrong cases. --- src/compiler/spirv/vtn_cfg.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/src/compiler/spirv/vtn_cfg.c b/src/compiler/spirv/vtn_cfg.c

[Mesa-dev] [PATCH] spirv: consider bitsize when handling OpSwitch cases

2017-12-19 Thread Juan A. Suarez Romero
When walking over all the cases in a OpSwitch, take in account the bitsize of the literals to avoid getting wrong cases. --- src/compiler/spirv/vtn_cfg.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/src/compiler/spirv/vtn_cfg.c b/src/compiler/spirv/vtn_cfg.c