I think that, after Zack's change, the generator programs that include
rtl.h should be linked with build/vec.o.  That may not be necessary when
optimizing, but it would avoid this problem.  Do you agree?
Well, if it fixes the bug, yes: I prefer to fix this in the makefile than with #ifdef GENERATOR_FILE's here and there.

An untested patch to do so is attached. You can try it and, if it fails, there is also Rainer Orth's patch in comment #14 of the PR.

Paolo
2006-06-13  Paolo Bonzini  <[EMAIL PROTECTED]>

        * Makefile.in (BUILD_RTL): Add build/vec.o.
        (build/genextract, build/genautomata): Remove dependency on it.

Index: Makefile.in
===================================================================
--- Makefile.in (revision 114487)
+++ Makefile.in (working copy)
@@ -854,7 +854,7 @@ LDEXP_LIB = @LDEXP_LIB@
 # even if we are cross-building GCC.
 BUILD_LIBS = $(BUILD_LIBIBERTY)
 
-BUILD_RTL = build/rtl.o build/read-rtl.o build/ggc-none.o \
+BUILD_RTL = build/rtl.o build/read-rtl.o build/ggc-none.o build/vec.o \
            build/min-insn-modes.o build/gensupport.o build/print-rtl.o
 BUILD_ERRORS = build/errors.o
 
@@ -3017,9 +3020,6 @@ genprogmd = attr attrtab automata codes 
 $(genprogmd:%=build/gen%$(build_exeext)): $(BUILD_RTL) $(BUILD_ERRORS)
 
 # These programs need files over and above what they get from the above list.
-build/genextract$(build_exeext) : build/vec.o
-
-build/genautomata$(build_exeext) : build/vec.o
 build/genautomata$(build_exeext) : BUILD_LIBS += -lm
 
 # These programs are not linked with the MD reader.

Reply via email to