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]

bootstrap broken on mainline for targets not using DFA


Mostafa Hagog wrote:

ChangeLog
2004-05-23  Ayal Zaks  <zaks@il.ibm.com>
      Mostafa Hagog  <mustafa@il.ibm.com>

      * Makefile.in (modulo-sched.o, ddg.o): New.
      * ddg.h, ddg.c, modulo-sched.c: New files.
      [...]


This patch breaks bootstap on the few remaining backends that
don't yet use the DFA scheduler, notably m68k:

stage1/xgcc -Bstage1/ -B/home/bernie/src/gcc-HEAD-install/m68k-unknown-linux-gnu/bin/ -c   -g -O2  -DIN_GCC   -W -Wall -Wwrite-strings -Wstrict-prototypes -Wmissing-prototypes -pedantic -Wno-long-long -Wno-variadic-macros -Wold-style-definition -Werror -fno-common   -DHAVE_CONFIG_H -I. -I. -I../../gcc-HEAD/gcc -I../../gcc-HEAD/gcc/. -I../../gcc-HEAD/gcc/../include -I../../gcc-HEAD/gcc/../libcpp/include -I../../gcc-HEAD/gcc/../libbanshee/libcompat -I../../gcc-HEAD/gcc/../libbanshee -I../../gcc-HEAD/gcc/../libbanshee/points-to  ../../gcc-HEAD/gcc/modulo-sched.c -o modulo-sched.o
../../gcc-HEAD/gcc/modulo-sched.c: In function `advance_one_cycle':
../../gcc-HEAD/gcc/modulo-sched.c:1988: warning: implicit declaration of function 'state_transition'
../../gcc-HEAD/gcc/modulo-sched.c: In function `ps_has_conflicts':
../../gcc-HEAD/gcc/modulo-sched.c:2011: warning: implicit declaration of function 'state_reset'
../../gcc-HEAD/gcc/modulo-sched.c:2030: warning: implicit declaration of function 'state_dead_lock_p'
make[3]: *** [modulo-sched.o] Error 1


This only happens during bootstraps (-Werror). Nothing from modulo-sched.o gets linked into cc1 because the part of invocation of sms_scedule() in passes.c is protected by "#ifdef INSN_SCHEDULE".

This patch fixed the problem for me. OK for mainline?


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


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


diff -u -p -U5 -r1.2 modulo-sched.c --- modulo-sched.c 30 May 2004 07:12:55 -0000 1.2 +++ modulo-sched.c 8 Jun 2004 20:10:05 -0000 @@ -18,10 +18,11 @@ for more details. You should have received a copy of the GNU General Public License along with GCC; see the file COPYING. If not, write to the Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */

+#ifdef INSN_SCHEDULING

#include "config.h"
#include "system.h"
#include "coretypes.h"
#include "tm.h"
@@ -2121,5 +2122,7 @@ 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/


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