Re: [patchset] rewrite of initializer handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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...
-
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux