Re: [Mesa-dev] [PATCH 1/2] mesa: remove redundant modulus operation

2017-05-23 Thread Timothy Arceri
On 24/05/17 10:25, Timothy Arceri wrote: > On 24/05/17 08:07, Ian Romanick wrote On 05/23/2017 05:01 AM, Timothy Arceri wrote: The if check above means we can only get here if size is less than 4. --- src/mesa/program/prog_parameter.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)

Re: [Mesa-dev] [PATCH 1/2] mesa: remove redundant modulus operation

2017-05-23 Thread Timothy Arceri
On 24/05/17 08:07, Ian Romanick wrote: On 05/23/2017 05:01 AM, Timothy Arceri wrote: The if check above means we can only get here if size is less than 4. --- src/mesa/program/prog_parameter.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/mesa/program/prog_par

Re: [Mesa-dev] [PATCH 1/2] mesa: remove redundant modulus operation

2017-05-23 Thread Ian Romanick
On 05/23/2017 05:01 AM, Timothy Arceri wrote: > The if check above means we can only get here if size is less than 4. > --- > src/mesa/program/prog_parameter.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/src/mesa/program/prog_parameter.c > b/src/mesa/program/pro

[Mesa-dev] [PATCH 1/2] mesa: remove redundant modulus operation

2017-05-23 Thread Timothy Arceri
The if check above means we can only get here if size is less than 4. --- src/mesa/program/prog_parameter.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/mesa/program/prog_parameter.c b/src/mesa/program/prog_parameter.c index 44e680c..40bc47d 100644 --- a/src/mesa/p