From: Peter Maydell <peter.maydell@linaro.org>
To: qemu-devel@nongnu.org
Cc: Blue Swirl <blauwirbel@gmail.com>, Aurelien Jarno <aurelien@aurel32.net>
Subject: [Qemu-devel] [PATCH 1/4] Makefile.target: Allow target helpers to be in any *_helper.c file
Date: Mon, 4 Apr 2011 11:46:32 +0100 [thread overview]
Message-ID: <1301913995-2506-2-git-send-email-peter.maydell@linaro.org> (raw)
In-Reply-To: <1301913995-2506-1-git-send-email-peter.maydell@linaro.org>
Build all files matching *_helper.c with HELPER_CFLAGS, not just
op_helper.c. This allows you to put target helper functions which
use the global 'env' variable in multiple source files.
This only affects the ARM target as all the other targets currently only
have op_helper.c.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
Makefile.target | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/Makefile.target b/Makefile.target
index 04e20dd..45acdba 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -94,7 +94,7 @@ tcg/tcg.o: cpu.h
# HELPER_CFLAGS is used for all the code compiled with static register
# variables
-op_helper.o cpu-exec.o: QEMU_CFLAGS += $(HELPER_CFLAGS)
+%_helper.o cpu-exec.o: QEMU_CFLAGS += $(HELPER_CFLAGS)
# Note: this is a workaround. The real fix is to avoid compiling
# cpu_signal_handler() in cpu-exec.c.
--
1.7.1
next prev parent reply other threads:[~2011-04-04 10:46 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-04 10:46 [Qemu-devel] [PATCH 0/4] target-arm: Make Neon helpers use correct FP status Peter Maydell
2011-04-04 10:46 ` Peter Maydell [this message]
2011-04-04 10:46 ` [Qemu-devel] [PATCH 2/4] target-arm: Use global env in neon_helper.c helpers Peter Maydell
2011-04-04 10:46 ` [Qemu-devel] [PATCH 3/4] target-arm: Use global env in iwmmxt_helper.c helpers Peter Maydell
2011-04-04 10:46 ` [Qemu-devel] [PATCH 4/4] target-arm: Make Neon helper routines use correct FP status Peter Maydell
2011-04-04 18:29 ` [Qemu-devel] Re: [PATCH 0/4] target-arm: Make Neon helpers " Aurelien Jarno
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=1301913995-2506-2-git-send-email-peter.maydell@linaro.org \
--to=peter.maydell@linaro.org \
--cc=aurelien@aurel32.net \
--cc=blauwirbel@gmail.com \
--cc=qemu-devel@nongnu.org \
/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;
as well as URLs for NNTP newsgroup(s).