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

Submitted by Alexander Graf on April 4, 2011, 2:32 p.m.

Details

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.
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.
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 hide | download patch | download mbox

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"