diff mbox

[02/15] s390x: set alignment for long to 8

Message ID 1301927544-32767-3-git-send-email-agraf@suse.de
State New
Headers show

Commit Message

Alexander Graf April 4, 2011, 2:32 p.m. UTC 
The alignment for longs on s390x is 8. That's the only place where it differs
from the default alignments found in configure already. The example alignment
program from Laurent printed the following on a real s390x:

  alignof(short) 2
  alignof(int) 4
  alignof(long) 8
  alignof(long long) 8

Signed-off-by: Alexander Graf <agraf@suse.de>
---
 configure |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

Aurelien Jarno April 10, 2011, 7:25 p.m. UTC | #1 
On Mon, Apr 04, 2011 at 04:32:11PM +0200, Alexander Graf wrote:
> The alignment for longs on s390x is 8. That's the only place where it differs
> from the default alignments found in configure already. The example alignment
> program from Laurent printed the following on a real s390x:
> 
>   alignof(short) 2
>   alignof(int) 4
>   alignof(long) 8
>   alignof(long long) 8
> 
> Signed-off-by: Alexander Graf <agraf@suse.de>
> ---
>  configure |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)

Thanks, applied.

> diff --git a/configure b/configure
> index faaed60..1e068e6 100755
> --- a/configure
> +++ b/configure
> @@ -3182,6 +3182,7 @@ case "$target_arch2" in
>    s390x)
>      target_nptl="yes"
>      target_phys_bits=64
> +    target_long_alignment=8
>    ;;
>    *)
>      echo "Unsupported target CPU"
> -- 
> 1.6.0.2
> 
> 
>
diff mbox

Patch

diff --git a/configure b/configure
index faaed60..1e068e6 100755
--- a/configure
+++ b/configure
@@ -3182,6 +3182,7 @@  case "$target_arch2" in
   s390x)
     target_nptl="yes"
     target_phys_bits=64
+    target_long_alignment=8
   ;;
   *)
     echo "Unsupported target CPU"