Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-03-03 Thread Jose Fonseca
On 02/03/15 20:03, Jason Ekstrand wrote: On Mon, Mar 2, 2015 at 8:02 AM, Jose Fonseca mailto:jfons...@vmware.com>> wrote: On 27/02/15 16:15, Brian Paul wrote: On 02/27/2015 09:10 AM, Ian Romanick wrote: On 02/26/2015 10:07 AM, Brian Paul wrote: On 02/

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-03-02 Thread Jason Ekstrand
On Mon, Mar 2, 2015 at 8:02 AM, Jose Fonseca wrote: > On 27/02/15 16:15, Brian Paul wrote: > >> On 02/27/2015 09:10 AM, Ian Romanick wrote: >> >>> On 02/26/2015 10:07 AM, Brian Paul wrote: >>> On 02/26/2015 09:51 AM, Jose Fonseca wrote: > This is to enable the code to build with -We

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-03-02 Thread Jose Fonseca
On 27/02/15 16:15, Brian Paul wrote: On 02/27/2015 09:10 AM, Ian Romanick wrote: On 02/26/2015 10:07 AM, Brian Paul wrote: On 02/26/2015 09:51 AM, Jose Fonseca wrote: This is to enable the code to build with -Werror=vla in the short term, and enable the code to build with MSVC2013 soon after.

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-03-02 Thread Jose Fonseca
On 27/02/15 22:50, Jason Ekstrand wrote: On Fri, Feb 27, 2015 at 6:04 AM, Jose Fonseca mailto:jfons...@vmware.com>> wrote: On 26/02/15 18:07, Brian Paul wrote: On 02/26/2015 09:51 AM, Jose Fonseca wrote: This is to enable the code to build with -Werror=vla in the

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-02-27 Thread Jason Ekstrand
On Fri, Feb 27, 2015 at 6:04 AM, Jose Fonseca wrote: > On 26/02/15 18:07, Brian Paul wrote: > >> On 02/26/2015 09:51 AM, Jose Fonseca wrote: >> >>> This is to enable the code to build with -Werror=vla in the short term, >>> and enable the code to build with MSVC2013 soon after. >>> --- >>> incl

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-02-27 Thread Kenneth Graunke
On Friday, February 27, 2015 02:04:47 PM Jose Fonseca wrote: > On 26/02/15 18:07, Brian Paul wrote: [snip] > > AFAIK, there's no zeroing version of alloca(). > > Yes, I also searched. And unfortunately inline functions can't be used > due to alloca semantics. I'm not sure if there's any C-prepr

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-02-27 Thread Ian Romanick
On 02/26/2015 10:07 AM, Brian Paul wrote: > On 02/26/2015 09:51 AM, Jose Fonseca wrote: >> This is to enable the code to build with -Werror=vla in the short term, >> and enable the code to build with MSVC2013 soon after. >> --- >> include/c99_alloca.h | 45 >> +

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-02-27 Thread Jose Fonseca
On 26/02/15 18:07, Brian Paul wrote: On 02/26/2015 09:51 AM, Jose Fonseca wrote: This is to enable the code to build with -Werror=vla in the short term, and enable the code to build with MSVC2013 soon after. --- include/c99_alloca.h | 45 s

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-02-27 Thread Kenneth Graunke
On Thursday, February 26, 2015 04:51:21 PM Jose Fonseca wrote: > This is to enable the code to build with -Werror=vla in the short term, > and enable the code to build with MSVC2013 soon after. > --- > include/c99_alloca.h | 45 > > src/glsl/ni

Re: [Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-02-26 Thread Brian Paul
On 02/26/2015 09:51 AM, Jose Fonseca wrote: This is to enable the code to build with -Werror=vla in the short term, and enable the code to build with MSVC2013 soon after. --- include/c99_alloca.h | 45 src/glsl/nir/nir_from_ssa.c |

[Mesa-dev] [PATCH 1/2] nir: Use alloca instead of variable length arrays.

2015-02-26 Thread Jose Fonseca
This is to enable the code to build with -Werror=vla in the short term, and enable the code to build with MSVC2013 soon after. --- include/c99_alloca.h | 45 src/glsl/nir/nir_from_ssa.c | 19 +++ src/glsl/nir/nir_live_variab