On Wed, 1 Mar 2017, Jakub Jelinek wrote:

> On Wed, Mar 01, 2017 at 01:08:58PM +0100, Richard Biener wrote:
> > 
> > In debugging a -Wuninitialized issue from ipa-cp.c which does
> > 
> >               vr.min = vr.max = wi::zero (INT_TYPE_SIZE);
> 
> Note maybe it would be faster to:
>       vr.min = wi::zero (INT_TYPE_SIZE);
>       vr.max = wi::zero (INT_TYPE_SIZE);
> 
> That doesn't mean your wide-int.h change isn't useful.

Note that rewriting like above doesn't fix the warning.  The issue
is from

generic_wide_int<storage>& generic_wide_int<T>::operator=(const T&) [with 
T = wi::hwi_with_prec; storage = wide_int_storage] (struct 
generic_wide_int * const this, const struct hwi_with_prec & x)
{
  struct wide_int_storage D.47458;
  struct generic_wide_int & D.52104;

  wide_int_storage::wide_int_storage<wi::hwi_with_prec> (&D.47458, x);
  try
    {
      this->D.16244 = D.47458;

where wide_int_storage::wide_int_storage<wi::hwi_with_prec> (&D.47458, x)
doesn't initialize all of wide_int_storage (but only up to len).

Richard.

Reply via email to