[Qemu-devel] [PATCH] qemu-common.h: Drop WORDS_ALIGNED define

[Qemu-devel] [PATCH] qemu-common.h: Drop WORDS_ALIGNED define

[Peter Maydell]

The WORDS_ALIGNED #define is not used anywhere, and hasn't been since
2013 when commit 612d590ebc6cef rewrote the various ld<type>_<endian>_p
functions to not use it. Remove the #define and the comment describing it.
Also remove the line in the comment about TARGET_WORDS_ALIGNED, since
it has never actually existed.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 include/exec/cpu-all.h | 5 -----
 include/qemu-common.h  | 4 ----
 2 files changed, 9 deletions(-)

diff --git a/include/exec/cpu-all.h b/include/exec/cpu-all.h
index 3911576..9f38edf 100644
--- a/include/exec/cpu-all.h
+++ b/include/exec/cpu-all.h
@@ -34,14 +34,9 @@
 
 /* some important defines:
  *
- * WORDS_ALIGNED : if defined, the host cpu can only make word aligned
- * memory accesses.
- *
  * HOST_WORDS_BIGENDIAN : if defined, the host cpu is big endian and
  * otherwise little endian.
  *
- * (TARGET_WORDS_ALIGNED : same for target cpu (not supported yet))
- *
  * TARGET_WORDS_BIGENDIAN : same for target cpu
  */
 
diff --git a/include/qemu-common.h b/include/qemu-common.h
index 835cbc6..1f2cb94 100644
--- a/include/qemu-common.h
+++ b/include/qemu-common.h
@@ -14,10 +14,6 @@
 
 #include "qemu/fprintf-fn.h"
 
-#if defined(__arm__) || defined(__sparc__) || defined(__mips__) || defined(__hppa__) || defined(__ia64__)
-#define WORDS_ALIGNED
-#endif
-
 #define TFR(expr) do { if ((expr) != -1) break; } while (errno == EINTR)
 
 #include "qemu/option.h"
-- 
1.9.1

Re: [Qemu-devel] [PATCH] qemu-common.h: Drop WORDS_ALIGNED define

[Michael Tokarev]

02.06.2016 17:05, Peter Maydell wrote:
> The WORDS_ALIGNED #define is not used anywhere, and hasn't been since
> 2013 when commit 612d590ebc6cef rewrote the various ld<type>_<endian>_p
> functions to not use it. Remove the #define and the comment describing it.
> Also remove the line in the comment about TARGET_WORDS_ALIGNED, since
> it has never actually existed.

Applied to -trivial, thanks!

/mjt