This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[patch, sim] Fix simulator Makefile
- From: "Steve Ellcey " <sellcey at mips dot com>
- To: <gdb-patches at gcc dot gnu dot org>
- Date: Fri, 22 Nov 2013 11:34:41 -0800
- Subject: [patch, sim] Fix simulator Makefile
- Authentication-results: sourceware.org; auth=none
I am trying to build a complete 32 bit toolchain, including the simulator,
on an x86-64 Linux box by setting CFLAGS and CXXFLAGS to '-O2 -g -m32'.
I ran into a problem when building the simulator because in the igen
subdirectory make is using -m32 when compiling the objects that go into igen
but not on the link line.
I tried setting LDFLAGS to -m32 but that did not help. I noticed that a
number of other targets in that Makefile (that no longer appear to be used)
use $(BUILD_CFLAGS) as well as $(BUILD_LDFLAGS) when linking. Making that
change when building igen fixed the problem.
This may be considered an obvious fix, but I thought I would send it out
first anyway just to double check.
OK to checkin?
Steve Ellcey
sellcey@mips.com
2013-11-22 Steve Ellcey <sellcey@mips.com>
* igen/Makefile.in (igen): Use BUILD_CFLAGS in link.
diff --git a/sim/igen/Makefile.in b/sim/igen/Makefile.in
index 7b2375c..8f6a42c 100644
--- a/sim/igen/Makefile.in
+++ b/sim/igen/Makefile.in
@@ -117,7 +117,7 @@ IGEN_OBJS=\
gen.o
igen: igen.o $(IGEN_OBJS)
- $(CC_FOR_BUILD) $(BUILD_LDFLAGS) -o igen igen.o $(IGEN_OBJS) $(LIBIBERTY_LIB)
+ $(CC_FOR_BUILD) $(BUILD_CFLAGS) $(BUILD_LDFLAGS) -o igen igen.o $(IGEN_OBJS) $(LIBIBERTY_LIB)
igen.o: igen.c misc.h filter_host.h lf.h table.h ld-decode.h ld-cache.h ld-insn.h filter.h gen-model.h gen-itable.h gen-icache.h gen-idecode.h gen-engine.h gen-semantics.h gen-support.h gen.h igen.h
$(CC_FOR_BUILD) $(BUILD_CFLAGS) -c $(srcdir)/igen.c