On Fri, Jul 31, 2015 at 07:53:16PM +0300, Ilya Verbin wrote:
> On Fri, Jul 31, 2015 at 19:27:58 +0300, Ilya Verbin wrote:
> > I've noticed that target MIC compiler from trunk hangs forever in
> > lto_input_mode_table in this loop, even on simple testcases.
> >
> > On Wed, Feb 18, 2015 at 11:00:35 +0100, Jakub Jelinek wrote:
> > + /* First search just the GET_CLASS_NARROWEST_MODE to wider modes,
> > + if not found, fallback to all modes. */
> > + int pass;
> > + for (pass = 0; pass < 2; pass++)
> > + for (machine_mode mr = pass ? VOIDmode
> > + : GET_CLASS_NARROWEST_MODE (mclass);
> > + pass ? mr < MAX_MACHINE_MODE : mr != VOIDmode;
> > + pass ? mr = (machine_mode) (m + 1)
> > + : mr = GET_MODE_WIDER_MODE (mr))
> > + if (GET_MODE_CLASS (mr) != mclass
> > + || GET_MODE_SIZE (mr) != size
> > + || GET_MODE_PRECISION (mr) != prec
> > + || GET_MODE_INNER (mr) != inner
> > + || GET_MODE_IBIT (mr) != ibit
> > + || GET_MODE_FBIT (mr) != fbit
> > + || GET_MODE_NUNITS (mr) != nunits)
> > + continue;
> >
> > Given that gomp-4_1-branch works ok, the problem was introduced somewhere
> > between 9 and 31 Jul. I'll try to find the revision.
>
> Shouldn't 'mr' be here instead of 'm'?
I think so. If it works, patch preapproved.
But wonder what changed that we haven't been triggering it before.
What mode do you think it on (mclass/size/prec/inner/ibit/fbit/nunits)?
Jakub
