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: [PATCH] check_cfg assert fix

From: Bernd Schmidt <bernds at codesourcery dot com>
To: Maxim Kuvyrkov <maxim at codesourcery dot com>
Cc: Steven Bosscher <stevenb dot gcc at gmail dot com>, Tom de Vries <vries at codesourcery dot com>, Vladimir Makarov <vmakarov at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Thu, 08 Sep 2011 20:54:34 +0200
Subject: Re: [PATCH] check_cfg assert fix
References: <4E65E463.6070004@codesourcery.com> <CABu31nOgd7X+E1Nfff9bAtOe=PoRz6MPi6u_59o-MqoXSdr1-w@mail.gmail.com> <E049876D-ED2D-42F4-AA58-0495536A1CEF@codesourcery.com> <CABu31nMsgQDvK1jWM=u-o0fWwDMuS3GFE3aFG9MR6C9xmjmzLw@mail.gmail.com> <CABu31nNDVNzu-_Yepg0Y2CVgu=O80N+tO1VHx29O+JpER=gzMg@mail.gmail.com> <316EBD5E-3D0C-423E-9DB6-26C5DBD6A532@codesourcery.com>

On 09/06/11 23:56, Maxim Kuvyrkov wrote:
> I agree.  I would rather remove the entirety of haifa-sched.c:
> check_cfg(); scheduler is not the right place for checking
> consistency of CFG.  Check_cfg() was useful for debugging scheduler
> patches, but now it is more of maintainance overhead.
> 
> Do I have a second vote for removal of check_cfg()?

I'd be OK with that. Saves me some time adapting it to some scheduler
patches I'll be submitting soon...


Bernd

Follow-Ups:
- Re: [PATCH] check_cfg assert fix
  - From: Maxim Kuvyrkov

References:
- [PATCH] check_cfg assert fix
  - From: Tom de Vries
- Re: [PATCH] check_cfg assert fix
  - From: Steven Bosscher
- Re: [PATCH] check_cfg assert fix
  - From: Maxim Kuvyrkov
- Re: [PATCH] check_cfg assert fix
  - From: Steven Bosscher
- Re: [PATCH] check_cfg assert fix
  - From: Steven Bosscher
- Re: [PATCH] check_cfg assert fix
  - From: Maxim Kuvyrkov

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