mbox series

[v2,0/2] configure: Define target access alignment in configure

Message ID 945056df7ff54ce6b272dce64721d916@tpw09926dag18e.domain1.systemhost.net
Headers show
Series configure: Define target access alignment in configure | expand

Message

Tony Nguyen July 22, 2019, 3:19 a.m. UTC 
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

Thanks to Aleksandar Markovic for the guide in v1 =)

Tony Nguyen (2):
  configure: Define TARGET_ALIGNED_ONLY earlier
  configure: Cosmetic yes to "yes" for consistency

 configure             | 12 ++++++++++--
 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, 18 insertions(+), 18 deletions(-)