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

login
register
mail settings
Submitter Alexander Graf
Date April 4, 2011, 2:32 p.m.
Message ID <1301927544-32767-3-git-send-email-agraf@suse.de>
Download mbox | patch
Permalink /patch/89632/
State New
Headers show

Comments

Alexander Graf - April 4, 2011, 2:32 p.m.
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(-)
Aurelien Jarno - April 10, 2011, 7:25 p.m.
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
> 
> 
>

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"