[v3] configure: Define target access alignment in configure

[Qemu-devel] [PATCH v3 0/1] configure: Define target access alignment in configure

tony.nguyen@bt.com posted 1 patch 4 years, 9 months ago

Diff against v1 v2 v4 v4
Download series mbox

Test docker-clang@ubuntu passed

Test s390x passed

Test asan passed

Test docker-mingw@fedora passed

Test FreeBSD passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/8a5c6ca782b64899b7d6fbf47e8e6869@tpw09926dag18e.domain1.systemhost.net

Maintainers: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Artyom Tarasenko <atar4qemu@gmail.com>, Aurelien Jarno <aurelien@aurel32.net>, Richard Henderson <rth@twiddle.net>, Aleksandar Markovic <amarkovic@wavecomp.com>, Max Filippov <jcmvbkbc@gmail.com>, Eduardo Habkost <ehabkost@redhat.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Aleksandar Rikalo <arikalo@wavecomp.com>

There is a newer version of this series

configure             | 10 +++++++++-
include/exec/poison.h |  1 +
include/qom/cpu.h     |  2 +-
target/alpha/cpu.h    |  2 --
target/hppa/cpu.h     |  1 -
target/mips/cpu.h     |  2 --
target/sh4/cpu.h      |  2 --
target/sparc/cpu.h    |  2 --
target/xtensa/cpu.h   |  2 --
tcg/tcg.c             |  2 +-
tcg/tcg.h             |  8 +++++---
11 files changed, 17 insertions(+), 17 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v3 0/1] configure: Define target access alignment in configure

Posted by tony.nguyen@bt.com 4 years, 9 months ago

Move the define of target access alignment earlier from
target/foo/cpu.h to configure.

Suggested in Richard Henderson's reply to "[PATCH 1/4] tcg: TCGMemOp
is now accelerator independent MemOp"

Analysed target/foo/cpu.h for more candidates to define earlier but             
did not spot any other straight forward predicates.

Possible future clean ups:
- TCG_GUEST_DEFAULT_MO and TCG_TARGET_DEFAULT_MO seems like duplicates
- TARGET_INSN_START_EXTRA_WORDS 1 seems redundant as ifndef value is 1

v2:
- split cosmetic changes into separate patch
- cc corresponding maintainers

v3:
- dropped cosmetic changes
- improved commit message

Tony Nguyen (1):
  configure: Define target access alignment in configure

 configure             | 10 +++++++++-
 include/exec/poison.h |  1 +
 include/qom/cpu.h     |  2 +-
 target/alpha/cpu.h    |  2 --
 target/hppa/cpu.h     |  1 -
 target/mips/cpu.h     |  2 --
 target/sh4/cpu.h      |  2 --
 target/sparc/cpu.h    |  2 --
 target/xtensa/cpu.h   |  2 --
 tcg/tcg.c             |  2 +-
 tcg/tcg.h             |  8 +++++---
 11 files changed, 17 insertions(+), 17 deletions(-)

-- 
1.8.3.1