On 4 April 2012 16:40, Peter Maydell <peter.mayd...@linaro.org> wrote:
> Having looked at configure I'm pretty sure what we want here is
> QEMU_CFLAGS="-march=i486 $QEMU_CFLAGS"
>
> because we're only doing this for the benefit of a particular bit
> of code in hw/vhost.c and so QEMU_CFLAGS is sufficient. Also this
> brings it into line with other places where we add a -march flag,
> which use QEMU_CFLAGS, not CFLAGS.
...and having dug around in the git history we find that QEMU_CFLAGS
were introduced in commit a558ee1, whose commit message defines the
difference like this:
QEMU_CFLAGS: flags without which we can't compile
CFLAGS: "-g -O2"
"-march=i486" is clearly "flags without which we can't compile",
so we should be setting it in QEMU_CFLAGS, not CFLAGS.
Olaf, if you want to submit a fixed patch I think we should apply
it to upstream qemu.
PS: remarks like
"This patch is against the qemu-xen tree, but it should apply also to
qemu.git since it has the same issue. Please apply to both trees."
should go below the '---' in a patch email so they don't appear
in the git commit message when the patch is applied.
thanks
-- PMM
