bootstrap broken on mainline for targets not using DFA

Bernardo Innocenti bernie@develer.com
Tue Jun 8 21:46:00 GMT 2004


Vladimir Makarov wrote:
> Richard Henderson wrote:
> 
>> On Tue, Jun 08, 2004 at 10:13:25PM +0200, Bernardo Innocenti wrote:
>>  
>>
>>> +#ifdef INSN_SCHEDULING
>>>
>>> #include "config.h"
>>>   
>>
>>
>> That's too early for INSN_SCHEDULING to ever be defined.
>>  
>>
> Ops...  Richard, thanks for the email.  I've missed this.  Bernado, 
> please ignore my previous email.  Ifdef should be at least after 
> sched-int.h.  Better to put if after all includes.  Otherwise, all ports 
> using a pipeline decription will be broken.

Silly me, I'm very ashamed :-)

This is what I finally committed:


2004-06-08  Bernardo Innocenti  <bernie@develer.com>

	* modulo-sched.c: Compile only when INSN_SCHEDULING is
	defined.

diff -u -p -r1.2 modulo-sched.c
--- modulo-sched.c	30 May 2004 07:12:55 -0000	1.2
+++ modulo-sched.c	8 Jun 2004 21:19:06 -0000
@@ -49,6 +49,7 @@ Software Foundation, 59 Temple Place - S
 #include "df.h"
 #include "ddg.h"
 
+#ifdef INSN_SCHEDULING
 
 /* This file contains the implementation of the Swing Modulo Scheduler,
    described in the following references:
@@ -2123,3 +2124,5 @@ rotate_partial_schedule (partial_schedul
   ps->max_cycle -= start_cycle;
   ps->min_cycle -= start_cycle;
 }
+
+#endif /* INSN_SCHEDULING*/

-- 
  // Bernardo Innocenti - Develer S.r.l., R&D dept.
\X/  http://www.develer.com/



More information about the Gcc mailing list