[PATCH] check_cfg assert fix

Maxim Kuvyrkov maxim@codesourcery.com
Tue Sep 13 20:45:00 GMT 2011


On 9/09/2011, at 6:54 AM, Bernd Schmidt wrote:

> 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...

OK then, attached is the trivial patch that removes haifa-sched.c:check_cfg().  Please let me know if you have strong feelings towards keeping check_cfg().

Tested on x86_64-linux-gnu.  Absent any requests to the contrary, I will check in this patch in 2 days.

Thank you,

--
Maxim Kuvyrkov
CodeSourcery / Mentor Graphics

-------------- next part --------------
A non-text attachment was scrubbed...
Name: fsf-gcc-check-cfg.ChangeLog
Type: application/octet-stream
Size: 193 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20110913/04626469/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fsf-gcc-check-cfg.patch
Type: application/octet-stream
Size: 4810 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20110913/04626469/attachment-0001.obj>


More information about the Gcc-patches mailing list