This is the mail archive of the gcc@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: New assert in haifa-sched.c

From: Eric Botcazou <ebotcazou at adacore dot com>
To: Maxim Kuvyrkov <maxim at codesourcery dot com>
Cc: gcc at gcc dot gnu dot org, Adam Nemet <anemet at caviumnetworks dot com>, abel at ispras dot ru
Date: Tue, 16 Sep 2008 21:45:06 +0200
Subject: Re: New assert in haifa-sched.c
References: <18624.50875.177784.531100@foo.home> <200809162040.52625.ebotcazou@adacore.com> <48D00481.9050102@codesourcery.com>

> I don't see the logic here.  It is certainly not that hard to fix the
> code to make this assertion trivial, simple loop through all insns in
> sched_init() will do it.

Yes, but apparently nobody is willing to do that at the moment so we'll need 
to kludge until then and add calls to recog_memoized when we stumble on insns 
that have not been recognized.  But our laziness shouldn't impact the user if 
we are confident that this cannot result in wrong code.

> There is a bug and it should get fixed rather than suppressed.

AFAICS nobody has submitted a real fix for this bug.  I'm ready to approve 
Adam's patch but only if the assertion is downgraded to ENABLE_CHECKING.

-- 
Eric Botcazou

References:
- New assert in haifa-sched.c
  - From: Adam Nemet
- Re: New assert in haifa-sched.c
  - From: Eric Botcazou
- Re: New assert in haifa-sched.c
  - From: Maxim Kuvyrkov

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