This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Jan Hubicka and Uros Bizjak appointed i386 maintainers


> > On Mon, 8 Jan 2007, David Edelsohn wrote:
> > > I am pleased to announce that the GCC Steering Committee has appointed 
> > > Jan Hubicka and Uros Bizjak as co-maintainers of the i386 port.
> > 
> > That's good timing. ;-)
> > 
> > i386 (but not i686) has started failing to bootstrap a few days ago
> > -- bootstrap/30467.
> 
> Hi,
> it is not my maintainance area, but the attached patch seems to fix it.
> The problem is that if conversion is converting conditional like:
> 
> if (val != mem)
>    val = 0;
> 
> For if conversion value a is 0, value b is MEM. insn_a is set to set to
> 0, but since MEM is derived from the conditional, insn_b is NULL.
> The conditional in question assumes insn_b is non-null and segfaults.  I
> think the proper fix is to simply check for B being NULL, but I
> dind'thad chance to follow in detail the recent ifcvt changes, so I
> guess Steven might want to comment.

Forgot to mention, other posibility is to make if conversion to set
insn_b to the conditional itself in such cases, but this is somewhat ill
defined for conditoinals spanning multiple insns and such.

Honza


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]