Experiment with changing the default 32-bit linux stack alignment to

16 bytes for PR8969. Update all testcases accordingly.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@123367 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Target/X86/X86Subtarget.cpp b/lib/Target/X86/X86Subtarget.cpp
index b72cae6a6f6f21796e80d8d3a597e9e03ed067fd..de768561f111ce90d03753bdd57ed6978f254f37 100644 (file)
--- a/lib/Target/X86/X86Subtarget.cpp
+++ b/lib/Target/X86/X86Subtarget.cpp
@@ -342,9 +342,9 @@ X86Subtarget::X86Subtarget(const std::string &TT, const std::string &FS,
   assert((!Is64Bit || HasX86_64) &&
          "64-bit code requested on a subtarget that doesn't support it!");
 
-  // Stack alignment is 16 bytes on Darwin (both 32 and 64 bit) and for all 64
-  // bit targets.
-  if (isTargetDarwin() || Is64Bit)
+  // Stack alignment is 16 bytes on Darwin and Linux (both 32 and 64 bit) and 
+  // for all 64-bit targets.
+  if (isTargetDarwin() || isTargetLinux() || Is64Bit)
     stackAlignment = 16;
 
   if (StackAlignment)

diff --git a/test/CodeGen/X86/2008-07-19-movups-spills.ll b/test/CodeGen/X86/2008-07-19-movups-spills.ll
index 98919ee5221ab5d34bd9c4011c99b46fe9e4fabb..cf04dcf0f18c88599a1bea35831d1ddbe0ed3bf3 100644 (file)
--- a/test/CodeGen/X86/2008-07-19-movups-spills.ll
+++ b/test/CodeGen/X86/2008-07-19-movups-spills.ll
@@ -1,6 +1,7 @@
 ; RUN: llc < %s -mtriple=i686-pc-linux -realign-stack=1 -mattr=sse2 | grep movaps | count 75
-; RUN: llc < %s -mtriple=i686-pc-linux -realign-stack=0 -mattr=sse2 | grep movaps | count 1
+; RUN: llc < %s -mtriple=i686-pc-linux -realign-stack=0 -mattr=sse2 | grep movaps | count 75
 ; PR2539
+; PR8969 - make 32-bit linux have a 16-byte aligned stack
 
 external global <4 x float>, align 1           ; <<4 x float>*>:0 [#uses=2]
 external global <4 x float>, align 1           ; <<4 x float>*>:1 [#uses=1]

diff --git a/test/CodeGen/X86/2008-10-27-StackRealignment.ll b/test/CodeGen/X86/2008-10-27-StackRealignment.ll
index 784bc72f42e90177b94e5e13a311827bb72fc768..3d0766cde8456f2b98f751b8601adb36390e6018 100644 (file)
--- a/test/CodeGen/X86/2008-10-27-StackRealignment.ll
+++ b/test/CodeGen/X86/2008-10-27-StackRealignment.ll
@@ -1,8 +1,8 @@
 ; Linux doesn't support stack realignment for functions with allocas (PR2888).
 ; Until it does, we shouldn't use movaps to access the stack.  On targets with
 ; sufficiently aligned stack (e.g. darwin) we should.
-
-; RUN: llc < %s -mtriple=i386-pc-linux-gnu -mcpu=yonah | not grep movaps
+; PR8969 - make 32-bit linux have a 16-byte aligned stack
+; RUN: llc < %s -mtriple=i386-pc-linux-gnu -mcpu=yonah | grep movaps | count 2
 ; RUN: llc < %s -mtriple=i686-apple-darwin9 -mcpu=yonah | grep movaps | count 2
 
 

diff --git a/test/CodeGen/X86/2009-08-06-inlineasm.ll b/test/CodeGen/X86/2009-08-06-inlineasm.ll
index de32c2159ce156225683e9ee4711ec5258b87974..f9b5f9e0b1fd1fe9b7af61dc00c8d461ea2e9297 100644 (file)
--- a/test/CodeGen/X86/2009-08-06-inlineasm.ll
+++ b/test/CodeGen/X86/2009-08-06-inlineasm.ll
@@ -1,10 +1,12 @@
-; RUN: llc -mtriple=i386-pc-linux-gnu < %s
+; RUN: false
+; XRUN: llc -mtriple=i386-pc-linux-gnu < %s
 ; PR4668
 ; XFAIL: *
 ; FIXME: If the coalescer happens to coalesce %level.1 with the copy to EAX
 ; (for ret) then this will fail to compile. The fundamental problem is
 ; once the coalescer fixes a virtual register to physical register we can't
-; evict it.
+; evict it. This started passing again due to the changes for PR8969
+; so I've disabled it with a bigger stick.
 
 define i32 @x(i32 %qscale) nounwind {
 entry:

diff --git a/test/CodeGen/X86/abi-isel.ll b/test/CodeGen/X86/abi-isel.ll
index 3ef312cb6596f50cbdfb94738af1f2d2c2518859..5b4d79fa22b9e2e7314685e177caebfc51c8ea54 100644 (file)
--- a/test/CodeGen/X86/abi-isel.ll
+++ b/test/CodeGen/X86/abi-isel.ll
@@ -8375,7 +8375,7 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: lcallee:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   x
 ; LINUX-32-STATIC-NEXT:        calll   x
 ; LINUX-32-STATIC-NEXT:        calll   x
@@ -8383,11 +8383,11 @@ entry:
 ; LINUX-32-STATIC-NEXT:        calll   x
 ; LINUX-32-STATIC-NEXT:        calll   x
 ; LINUX-32-STATIC-NEXT:        calll   x
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: lcallee:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   x
 ; LINUX-32-PIC-NEXT:   calll   x
 ; LINUX-32-PIC-NEXT:   calll   x
@@ -8395,7 +8395,8 @@ entry:
 ; LINUX-32-PIC-NEXT:   calll   x
 ; LINUX-32-PIC-NEXT:   calll   x
 ; LINUX-32-PIC-NEXT:   calll   x
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: lcallee:
@@ -8506,7 +8507,7 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: dcallee:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   y
 ; LINUX-32-STATIC-NEXT:        calll   y
 ; LINUX-32-STATIC-NEXT:        calll   y
@@ -8514,11 +8515,11 @@ entry:
 ; LINUX-32-STATIC-NEXT:        calll   y
 ; LINUX-32-STATIC-NEXT:        calll   y
 ; LINUX-32-STATIC-NEXT:        calll   y
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: dcallee:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   y
 ; LINUX-32-PIC-NEXT:   calll   y
 ; LINUX-32-PIC-NEXT:   calll   y
@@ -8526,7 +8527,8 @@ entry:
 ; LINUX-32-PIC-NEXT:   calll   y
 ; LINUX-32-PIC-NEXT:   calll   y
 ; LINUX-32-PIC-NEXT:   calll   y
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: dcallee:
@@ -8770,17 +8772,18 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: caller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   callee
 ; LINUX-32-STATIC-NEXT:        calll   callee
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: caller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   callee
 ; LINUX-32-PIC-NEXT:   calll   callee
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: caller:
@@ -8844,17 +8847,18 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: dcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   dcallee
 ; LINUX-32-STATIC-NEXT:        calll   dcallee
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: dcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   dcallee
 ; LINUX-32-PIC-NEXT:   calll   dcallee
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: dcaller:
@@ -8918,17 +8922,18 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: lcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   lcallee
 ; LINUX-32-STATIC-NEXT:        calll   lcallee
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: lcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   lcallee
 ; LINUX-32-PIC-NEXT:   calll   lcallee
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: lcaller:
@@ -8990,15 +8995,16 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: tailcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   callee
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: tailcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   callee
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: tailcaller:
@@ -9053,15 +9059,16 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: dtailcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   dcallee
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: dtailcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   dcallee
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: dtailcaller:
@@ -9116,15 +9123,16 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: ltailcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   lcallee
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: ltailcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   lcallee
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: ltailcaller:
@@ -9183,17 +9191,18 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: icaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   *ifunc
 ; LINUX-32-STATIC-NEXT:        calll   *ifunc
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: icaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   *ifunc
 ; LINUX-32-PIC-NEXT:   calll   *ifunc
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: icaller:
@@ -9272,17 +9281,18 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: dicaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   *difunc
 ; LINUX-32-STATIC-NEXT:        calll   *difunc
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: dicaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   *difunc
 ; LINUX-32-PIC-NEXT:   calll   *difunc
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: dicaller:
@@ -9354,17 +9364,18 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: licaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   *lifunc
 ; LINUX-32-STATIC-NEXT:        calll   *lifunc
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: licaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   *lifunc
 ; LINUX-32-PIC-NEXT:   calll   *lifunc
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: licaller:
@@ -9435,17 +9446,18 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: itailcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   *ifunc
 ; LINUX-32-STATIC-NEXT:        calll   *ifunc
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: itailcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   *ifunc
 ; LINUX-32-PIC-NEXT:   calll   *ifunc
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: itailcaller:
@@ -9521,15 +9533,16 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: ditailcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   *difunc
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: ditailcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   *difunc
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: ditailcaller:
@@ -9588,15 +9601,16 @@ entry:
 ; LINUX-64-STATIC: ret
 
 ; LINUX-32-STATIC: litailcaller:
-; LINUX-32-STATIC:     pushl
+; LINUX-32-STATIC:     subl
 ; LINUX-32-STATIC-NEXT:        calll   *lifunc
-; LINUX-32-STATIC-NEXT:        popl
+; LINUX-32-STATIC-NEXT:        addl
 ; LINUX-32-STATIC-NEXT:        ret
 
 ; LINUX-32-PIC: litailcaller:
-; LINUX-32-PIC:        pushl
+; LINUX-32-PIC:        subl
 ; LINUX-32-PIC-NEXT:   calll   *lifunc
-; LINUX-32-PIC-NEXT:   popl
+; LINUX-32-PIC-NEXT:   addl
+
 ; LINUX-32-PIC-NEXT:   ret
 
 ; LINUX-64-PIC: litailcaller:

diff --git a/test/CodeGen/X86/sse-align-11.ll b/test/CodeGen/X86/sse-align-11.ll
index 3cc83ca0db2a0ca6ccd6bf3958cb7a464a1132f7..9f5d4b40d61a0f48e867f4246f420483eea58417 100644 (file)
--- a/test/CodeGen/X86/sse-align-11.ll
+++ b/test/CodeGen/X86/sse-align-11.ll
@@ -1,5 +1,6 @@
 ; RUN: llc < %s -march=x86 -mcpu=yonah -mtriple=i686-apple-darwin8 | grep movaps
-; RUN: llc < %s -march=x86 -mcpu=yonah -mtriple=i686-linux-gnu | grep movups
+; RUN: llc < %s -march=x86 -mcpu=yonah -mtriple=i686-linux-gnu | grep movaps
+; PR8969 - make 32-bit linux have a 16-byte aligned stack
 
 define <4 x float> @foo(float %a, float %b, float %c, float %d) nounwind {
 entry: