This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: [tree-ssa] Bootstrap failure on alpha
> Jan Hubicka <jh@suse.cz> writes:
>
> >>
> >> Jan,
> >>
> >> One of these two patches introduced a bootstrap regression in alpha.
> >> -Werror is triggered for the stage1 compiler when building cppcharset.c
> >
> > This is the same problem as in other two warnings. The function
> > one_utf8_to_cppchar takes s as reference. This used to count as
> > initialization, but now we inline the function and we realize that there
> > is path trought the function not initializing s. I will commit the
> > patch to iniitialize s to 0 as obivous to both mainline and branch as we
> > did for the other case.
>
> The warning is spurious -- if one_utf8_to_cppchar returns 0 then s has
> been initialized; if it returns any other value, s is not used. I
> have no objection to adding the default initialization, but you might
> want to think about making the warning more accurate.
We've discussed this with Jeff as the other cases has been same.
Our uninitialized warning has beem always CFG based and doing more
acurate dataflow is probably not affordable.
I am not sure whether in the case of inlining we don't want to shup up
the warning completely. I in general don't like the idea of warnings
depending on optimization decisions. But at the moment it is not doable
as warning comes from RTL backend still.
Honza
>
> zw