Re: [patchset] rewrite of initializer handling

[Al Viro <viro@ftp.linux.org.uk>]

On Mon, Jun 18, 2007 at 11:46:13PM +0200, Michael Stefaniuc wrote:

> > Umm...  I don't think that it's related.  count_array_initializer() would
> > work just fine for that one, since the straightforward list element counting
> > would work as-is.
> Right, it doesn't fixes this error. Actually the output of
> make clean; CHECK='sparse -Wno-transparent-union -Wno-old-initializer
> -Wno-non-pointer-null' make CC=cgcc > build.out 2>&1
> is identical bit by bit with or without Al's patch series.

Eh...  count_array_initializer() is not even touched in that series anyway.
The question had been how to fix it; Josh asked if fixing it would take
care of your report as well and I said "no".  Patch that followed was
unrelated to count_array_initializer() problems or problems fixed by
patch series that went into the tree.

> Works for Wine and removes most of the "error: cannot size expression"
> errors.

Any regressions?

> The 7 remaining errors are preceded by "error: bad constant
> expression" in the same line. Have to condense those into a test case too.

Probably genuine VLAs...