[Piglit] [PATCH 2/3] Use alloca instead of variable length arrays
idr at freedesktop.org
Mon Feb 9 11:17:59 PST 2015
On 02/06/2015 11:46 AM, Jose Fonseca wrote:
> I haven't tried this sort of code with MSVC 2013 U4, but at least in the
> past, MSVC 2013 was refusing certain kinds of variable length arrays.
> And Jan's patch is a stepping stone to -Wvla option when compiling
> (option which apply to the whole tree, including parts that are not
> built with MSVC), in order to preemptively spot uses of VLA in places
> where MSVC would break.
In Mesa, we can change compiler options in subdirectories. Is it
possible to do this with cmake? Like, could we remove -Wvla in
directories that are only built on Linux? It seems like that could give
us the best of both worlds...
> So let me do the following: I'll see if MSVC 2013 accepts or refuses the
> VLA code that caused problems the first time, and then we'll take it
> from there.
More information about the Piglit