Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: Richard Earnshaw <Richard.Earnshaw@foss.arm.com>, sellcey@imgtec.com
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH] sim: move sim-engine.o/sim-hrw.o to the common list
Date: Sun, 12 Apr 2015 09:54:00 -0000	[thread overview]
Message-ID: <20150412095422.GH16816@vapier> (raw)
In-Reply-To: <55250C69.8030100@foss.arm.com>

[-- Attachment #1: Type: text/plain, Size: 1115 bytes --]

On 08 Apr 2015 12:09, Richard Earnshaw wrote:
> On 07/04/15 04:57, Mike Frysinger wrote:
> > This makes these two objects available to all sims by default.
> 
> Which breaks the build for the ARM simulator:

odd, it's building & passing tests for me.  i usually check every sim target 
before committing.

at any rate, it's clearly not working for you, so i'll try to fix it for 
arm/mips thusly.
-mike

--- a/sim/arm/Makefile.in
+++ b/sim/arm/Makefile.in
@@ -20,11 +20,12 @@
 SIM_EXTRA_CFLAGS = -DMODET
 
 SIM_OBJS = \
+	wrapper.o \
 	$(SIM_NEW_COMMON_OBJS) \
 	sim-cpu.o \
 	sim-hload.o \
 	armemu26.o armemu32.o arminit.o armos.o armsupp.o \
-	armvirt.o bag.o thumbemu.o wrapper.o \
+	armvirt.o bag.o thumbemu.o \
 	armcopro.o maverick.o iwmmxt.o
 
 ## COMMON_POST_CONFIG_FRAG
--- a/sim/mips/Makefile.in
+++ b/sim/mips/Makefile.in
@@ -40,10 +40,10 @@ SIM_MULTI_OBJ = itable.o @sim_multi_obj@
 MIPS_EXTRA_LIBS = @mips_extra_libs@
 
 SIM_OBJS = \
+	interp.o \
 	$(SIM_@sim_gen@_OBJ) \
 	$(SIM_NEW_COMMON_OBJS) \
 	cp1.o \
-	interp.o \
 	mdmx.o \
 	dsp.o \
 	sim-main.o \

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  parent reply	other threads:[~2015-04-12  9:54 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-07  3:57 Mike Frysinger
2015-04-08 11:09 ` Richard Earnshaw
2015-04-08 15:46   ` Steve Ellcey
2015-04-12  9:54   ` Mike Frysinger [this message]
2015-04-09  6:42 ` Bin.Cheng
2015-04-10  9:05 ` Yao Qi
2015-04-12  9:55   ` Mike Frysinger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150412095422.GH16816@vapier \
    --to=vapier@gentoo.org \
    --cc=Richard.Earnshaw@foss.arm.com \
    --cc=gdb-patches@sourceware.org \
    --cc=sellcey@imgtec.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox