[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 0/4] build: TARGET_ARCH/ARCH2/TYPE simplific
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [RFC PATCH 0/4] build: TARGET_ARCH/ARCH2/TYPE simplification |
Date: |
Mon, 20 May 2013 22:57:45 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 |
Am 20.05.2013 19:23, schrieb Paolo Bonzini:
> We have three variables currently in config-target.h:
>
> - TARGET_ARCH is used to create a unique per-arch symbol, used in #ifdefs.
> It is also used as a string through config-target.h, but this is almost
> always wrong.
>
> - TARGET_ARCH2 is the name of the executable (minus the qemu-/qemu-system-
> prefix); it is not available in config-target.h.
>
> - TARGET_TYPE is an enum but is otherwise the same as TARGET_ARCH2
Add to that
- TARGET_BASE_ARCH in Makefile.target is assumed to be the name of the
target-* subdirectory, but this will not always be desirable.
> This series changes all uses of TARGET_ARCH to refer to TARGET_ARCH2
> instead (which is renamed to TARGET_NAME). The TARGET_ARCH #define
> is dropped, only the per-arch symbol remains. TARGET_TYPE is then also
> removed since it is serialized to the same string if TARGET_NAME is
> used directly.
For a long time already I have been carrying the following patch
introducing TARGET_ARCH_DIR:
https://github.com/afaerber/qemu-rl78/commit/9eddd60dae6603235fddf9070c73e5bf65edf292
Still need to rebase that, but it looks as if there are no conflicts
with your series.
TARGET_NAME while a bit invasive is certainly nicer than TARGET_ARCH2,
and dropping TargetType enum makes rebasing/adding new targets easier.
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg