[RFC] Rearranging the build order fix link problem in tmp-ld-insn

[RFC] Rearranging the build order fix link problem in tmp-ld-insn

[Manoj Iyer]

When compiling/linking tmp-ld-insn, the linker exits with the error
message, cc -g -O -I. -I. -I./../../include -I../../bfd -I./../../bfd
-I../../gdb -I./../../gdb  -I./../../gdb/config   -o tmp-ld-insn -DMAIN
./ld-insn.c ld-cache.o misc.o lf.o table.o ld-decode.o filter_host.o
filter.o
cc: 1501-228 input file ld-cache.o not found
cc: 1501-228 input file ld-decode.o not found
Rearranging the build order fixes this build break. Here is the patch.

Please review and ok to commit.

=========================================================================
2005-04-21  Manoj Iyer  <manjo@austin.ibm.com>

        * Rearranging the build order in sim/ppc/Makefile.in to fix build
        break reported by AIX linker.

--- ./src/sim/ppc/Makefile.in	2005-01-03 09:34:19.000000000 -0600
+++ ./new/src/sim/ppc/Makefile.in	2005-04-21 14:12:56.000000000 -0500
@@ -683,7 +683,7 @@ tmp-dgen: dgen ppc-spr-table $(srcdir)/.
 	$(SHELL) $(srcdir)/../../move-if-change tmp-spreg.c spreg.c
 	touch tmp-dgen

-tmp-igen: igen $(srcdir)/ppc-instructions $(srcdir)/altivec.igen $(srcdir)/e500.igen $(IGEN_OPCODE_RULES) $(srcdir)/../../move-if-change tmp-ld-decode tmp-ld-cache tmp-ld-insn tmp-filter
+tmp-igen: igen $(srcdir)/ppc-instructions $(srcdir)/altivec.igen $(srcdir)/e500.igen $(IGEN_OPCODE_RULES) $(srcdir)/../../move-if-change tmp-ld-insn tmp-ld-decode tmp-ld-cache tmp-filter
 	./igen	$(IGEN_FLAGS) \
 		-o $(srcdir)/$(IGEN_OPCODE_RULES) \
 		-I $(srcdir) -i $(srcdir)/ppc-instructions \

Thanks
-----
manjo
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+ Cogito ergo sum                                                          +
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Re: [RFC] Rearranging the build order fix link problem in tmp-ld-insn

[Daniel Jacobowitz]

On Thu, Apr 21, 2005 at 02:20:26PM -0500, Manoj Iyer wrote:
> 
> When compiling/linking tmp-ld-insn, the linker exits with the error
> message, cc -g -O -I. -I. -I./../../include -I../../bfd -I./../../bfd
> -I../../gdb -I./../../gdb  -I./../../gdb/config   -o tmp-ld-insn -DMAIN
> ./ld-insn.c ld-cache.o misc.o lf.o table.o ld-decode.o filter_host.o
> filter.o
> cc: 1501-228 input file ld-cache.o not found
> cc: 1501-228 input file ld-decode.o not found
> Rearranging the build order fixes this build break. Here is the patch.

Please take another look at my response the first time you posted this.
I wrote:

  #2 is definitely wrong; there are missing dependencies somewhere. 
  The order of things in a dependency line is not significant to make
  except accidentally.

> 2005-04-21  Manoj Iyer  <manjo@austin.ibm.com>
> 
>         * Rearranging the build order in sim/ppc/Makefile.in to fix build
>         break reported by AIX linker.

Please pay more attention to your ChangeLog entries.

	* Makefile.in (tmp-igen): Rearrange the order of dependencies.

-- 
Daniel Jacobowitz
CodeSourcery, LLC