On Sat, Apr 8, 2017 at 2:18 PM, Luc Van Oostenryck
<luc.vanoostenryck@xxxxxxxxx> wrote:
> diff --git a/linearize.c b/linearize.c
> index 5c273a91d..f628f5a74 100644
> --- a/linearize.c
> +++ b/linearize.c
> @@ -1635,6 +1635,21 @@ static pseudo_t linearize_one_symbol(struct entrypoint *ep, struct symbol *sym)
>
> sym->initialized = 1;
> ad.address = symbol_pseudo(ep, sym);
> +
> + if (sym->initializer && !is_scalar_type(sym)) {
> + // default zero initialization [6.7.9.21]
> + // FIXME: this init the whole aggregate while
> + // only the existing fields need to be initialized.
> + // FIXME: this init the whole aggregate even if
> + // all fields arelater explicitely initialized.
Some very minor comment. The multi-line comment should be
written in the C type. /* ... */
Chris
--
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
