Re: [Mesa-dev] [PATCH 7/8] spirv: Restructure the case loop in OpSwitch handling

2017-12-11 Thread Ian Romanick
On 12/11/2017 02:50 PM, Jason Ekstrand wrote: > On Mon, Dec 11, 2017 at 10:08 AM, Ian Romanick > wrote: > > On 12/07/2017 08:12 AM, Jason Ekstrand wrote: > > Instead of calling vtn_add_case for the default case and then looping, > >

Re: [Mesa-dev] [PATCH 7/8] spirv: Restructure the case loop in OpSwitch handling

2017-12-11 Thread Jason Ekstrand
On Mon, Dec 11, 2017 at 10:08 AM, Ian Romanick wrote: > On 12/07/2017 08:12 AM, Jason Ekstrand wrote: > > Instead of calling vtn_add_case for the default case and then looping, > > add an is_default variable and do everything inside the loop. This will > > make the next

Re: [Mesa-dev] [PATCH 7/8] spirv: Restructure the case loop in OpSwitch handling

2017-12-11 Thread Ian Romanick
On 12/07/2017 08:12 AM, Jason Ekstrand wrote: > Instead of calling vtn_add_case for the default case and then looping, > add an is_default variable and do everything inside the loop. This will > make the next commit easier. > --- > src/compiler/spirv/vtn_cfg.c | 17 ++--- > 1 file

[Mesa-dev] [PATCH 7/8] spirv: Restructure the case loop in OpSwitch handling

2017-12-07 Thread Jason Ekstrand
Instead of calling vtn_add_case for the default case and then looping, add an is_default variable and do everything inside the loop. This will make the next commit easier. --- src/compiler/spirv/vtn_cfg.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git