diff options
author | Artyom Kunakovsky <artyomkunakovsky@gmail.com> | 2024-05-23 08:11:18 +0300 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2024-05-24 09:27:56 +0200 |
commit | c2bf2ccb266dc9ae4a6da75b845f54535417e109 (patch) | |
tree | c168c5f4bbf70434b09e0fc408ef721c6bc92bdb /qemu-keymap.c | |
parent | 70581940cabcc51b329652becddfbc6a261b1b83 (diff) | |
download | qemu-c2bf2ccb266dc9ae4a6da75b845f54535417e109.zip qemu-c2bf2ccb266dc9ae4a6da75b845f54535417e109.tar.gz qemu-c2bf2ccb266dc9ae4a6da75b845f54535417e109.tar.bz2 |
configure: move -mcx16 flag out of CPU_CFLAGS
The point of CPU_CFLAGS is really just to select the appropriate multilib,
for example for library linking tests, and -mcx16 is not needed for
that purpose.
Furthermore, if -mcx16 is part of QEMU's choice of a basic x86_64
instruction set, it should be applied to cross-compiled x86_64 code too;
it is plausible that tests/tcg would want to cover cmpxchg16b as well,
for example. In the end this makes just as much sense as a per sub-build
tweak, so move the flag to meson.build and cross_cc_cflags_x86_64.
This leaves out contrib/plugins, which would fail when attempting to use
__sync_val_compare_and_swap_16 (note it does not do yet); while minor,
this *is* a disadvantage of this change. But building contrib/plugins
with a Makefile instead of meson.build is something self-inflicted just
for the sake of showing that it can be done, and if this kind of papercut
started becoming a problem we could make the directory part of the meson
build. Until then, we can live with the limitation.
Signed-off-by: Artyom Kunakovsky <artyomkunakovsky@gmail.com>
Message-ID: <20240523051118.29367-1-artyomkunakovsky@gmail.com>
[rewrite commit message, remove from configure. - Paolo]
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'qemu-keymap.c')
0 files changed, 0 insertions, 0 deletions