[PATCH 0/2] S/390 test case fixes.

[PATCH 0/2] S/390 test case fixes.

[Dominik Vogt]

The following two patches fix some test cases for S/390.

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany

Re: [PATCH 0/2] S/390 test case fixes.

[Dominik Vogt]

[-- Attachment #1: Type: text/plain, Size: 256 bytes --]

On Mon, Jan 11, 2016 at 11:46:21AM +0100, Dominik Vogt wrote:
> The following two patches fix some test cases for S/390.

Patch 1:

  gcc.dg/tree-ssa/20040204-1.c does not fail on S/390 at the moment.

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany

[-- Attachment #2: 0001-ChangeLog --]
[-- Type: text/plain, Size: 81 bytes --]

gcc/testsuite/ChangeLog

	* gcc.dg/tree-ssa/20040204-1.c: Do not xfail on S/390.

[-- Attachment #3: 0001-S-390-gcc.dg-tree-ssa-20040204-1.c-Remove-xfail.patch --]
[-- Type: text/x-diff, Size: 1326 bytes --]

From 662b6943ad387db3099398ef4726c81afdd69709 Mon Sep 17 00:00:00 2001
From: Dominik Vogt <vogt@linux.vnet.ibm.com>
Date: Fri, 8 Jan 2016 13:31:02 +0100
Subject: [PATCH 1/2] S/390: gcc.dg/tree-ssa/20040204-1.c: Remove xfail.

---
 gcc/testsuite/gcc.dg/tree-ssa/20040204-1.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/testsuite/gcc.dg/tree-ssa/20040204-1.c b/gcc/testsuite/gcc.dg/tree-ssa/20040204-1.c
index de2ae0f..f7b5dfa 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/20040204-1.c
+++ b/gcc/testsuite/gcc.dg/tree-ssa/20040204-1.c
@@ -33,4 +33,4 @@ void test55 (int x, int y)
    that the && should be emitted (based on BRANCH_COST).  Fix this
    by teaching dom to look through && and register all components
    as true.  */
-/* { dg-final { scan-tree-dump-times "link_error" 0 "optimized" { xfail { ! "alpha*-*-* arm*-*-* aarch64*-*-* powerpc*-*-* cris-*-* crisv32-*-* hppa*-*-* i?86-*-* mmix-*-* mips*-*-* m68k*-*-* moxie-*-* nds32*-*-* sh*-*-* sparc*-*-* spu-*-* visium-*-* x86_64-*-*" } } } } */
+/* { dg-final { scan-tree-dump-times "link_error" 0 "optimized" { xfail { ! "alpha*-*-* arm*-*-* aarch64*-*-* powerpc*-*-* cris-*-* crisv32-*-* hppa*-*-* i?86-*-* mmix-*-* mips*-*-* m68k*-*-* moxie-*-* nds32*-*-* s390*-*-* sh*-*-* sparc*-*-* spu-*-* visium-*-* x86_64-*-*" } } } } */
-- 
2.3.0

Re: [PATCH 0/2] S/390 test case fixes.

[Dominik Vogt]

[-- Attachment #1: Type: text/plain, Size: 261 bytes --]

On Mon, Jan 11, 2016 at 11:46:21AM +0100, Dominik Vogt wrote:
> The following two patches fix some test cases for S/390.

Patch 2:

  Add -mbackchain on S/390 to make the -W[no-]frame-adrress tests
  work.

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany

[-- Attachment #2: 0002-ChangeLog --]
[-- Type: text/plain, Size: 131 bytes --]

gcc/testsuite/ChangeLog

	* g++.dg/Wframe-address.C: Test requires -mbackchain for s390*-*-*
	* g++.dg/Wno-frame-address.C: Ditto.

[-- Attachment #3: 0002-S-390-Add-mbackchain-for-W-no-frame-address-tests.patch --]
[-- Type: text/x-diff, Size: 1347 bytes --]

From 624f53f8b6fbbe3dfd70b86ebbdec053eff0830f Mon Sep 17 00:00:00 2001
From: Dominik Vogt <vogt@linux.vnet.ibm.com>
Date: Mon, 11 Jan 2016 11:35:07 +0100
Subject: [PATCH 2/2] S/390: Add -mbackchain for -W[no-]frame-address tests.

---
 gcc/testsuite/g++.dg/Wframe-address.C    | 1 +
 gcc/testsuite/g++.dg/Wno-frame-address.C | 1 +
 2 files changed, 2 insertions(+)

diff --git a/gcc/testsuite/g++.dg/Wframe-address.C b/gcc/testsuite/g++.dg/Wframe-address.C
index 229004e..849cf38 100644
--- a/gcc/testsuite/g++.dg/Wframe-address.C
+++ b/gcc/testsuite/g++.dg/Wframe-address.C
@@ -1,5 +1,6 @@
 // { dg-do compile }
 // { dg-options "-Wframe-address" }
+// { dg-additional-options "-mbackchain" { target s390*-*-* } }
 
 static void* const fa[] = {
   __builtin_frame_address (0),
diff --git a/gcc/testsuite/g++.dg/Wno-frame-address.C b/gcc/testsuite/g++.dg/Wno-frame-address.C
index 32a5cfd..a0e9c36 100644
--- a/gcc/testsuite/g++.dg/Wno-frame-address.C
+++ b/gcc/testsuite/g++.dg/Wno-frame-address.C
@@ -1,6 +1,7 @@
 // { dg-do compile }
 // { dg-skip-if "Cannot access arbitrary stack frames." { arm*-*-* hppa*-*-* } }
 // { dg-options "-Werror" }
+// { dg-additional-options "-mbackchain" { target s390*-*-* } }
 
 // Verify that -Wframe-address is not enabled by default by enabling
 // -Werror and verifying the test still compiles.
-- 
2.3.0

Re: [PATCH 3/2] S/390 test case fixes.

[Dominik Vogt]

[-- Attachment #1: Type: text/plain, Size: 299 bytes --]

On Mon, Jan 11, 2016 at 11:46:21AM +0100, Dominik Vogt wrote:
> The following two patches fix some test cases for S/390.

Patch 3:

  Disable g++.dg/init/const9.C on S/390 as discussed here:
  https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56194

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany

[-- Attachment #2: 0001-ChangeLog --]
[-- Type: text/plain, Size: 87 bytes --]

gcc/testsuite/ChangeLog

	PR c++/56194
	* g++.dg/init/const9.C: Disable test on S/390.

[-- Attachment #3: 0001-S-390-PR-56194-Disable-test-g-.dg-init-const9.patch --]
[-- Type: text/x-diff, Size: 736 bytes --]

From 0256f2d12a856306c6a4ae63a4898ad086f597d2 Mon Sep 17 00:00:00 2001
From: Dominik Vogt <vogt@linux.vnet.ibm.com>
Date: Mon, 11 Jan 2016 13:26:04 +0100
Subject: [PATCH] S/390: PR/56194: Disable test g++.dg/init/const9

---
 gcc/testsuite/g++.dg/init/const9.C | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/testsuite/g++.dg/init/const9.C b/gcc/testsuite/g++.dg/init/const9.C
index d733e95..648bf8d 100644
--- a/gcc/testsuite/g++.dg/init/const9.C
+++ b/gcc/testsuite/g++.dg/init/const9.C
@@ -1,5 +1,5 @@
 // PR c++/55893
-// { dg-final { scan-assembler-not "rodata" { target { ! hppa*-*-* } } } }
+// { dg-final { scan-assembler-not "rodata" { target { ! hppa*-*-* s390*-*-* } } } }
 
 struct foo
 {
-- 
2.3.0

Re: [PATCH 3/2] S/390 test case fixes.

[Dominik Vogt]

[-- Attachment #1: Type: text/plain, Size: 489 bytes --]

On Mon, Jan 11, 2016 at 01:28:36PM +0100, Dominik Vogt wrote:
> On Mon, Jan 11, 2016 at 11:46:21AM +0100, Dominik Vogt wrote:
> > The following two patches fix some test cases for S/390.
> 
> Patch 3:
> 
>   Disable g++.dg/init/const9.C on S/390 as discussed here:
>   https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56194

Sorry, the previous patch had a syntax error that I did not notice
when running the test.  New version attached.

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany

[-- Attachment #2: 0001-ChangeLog --]
[-- Type: text/plain, Size: 87 bytes --]

gcc/testsuite/ChangeLog

	PR c++/56194
	* g++.dg/init/const9.C: Disable test on S/390.

[-- Attachment #3: 0001-S-390-PR-56194-Disable-test-g-.dg-init-const9.patch --]
[-- Type: text/x-diff, Size: 738 bytes --]

From 630effb5e1afadc9836f23588a5278ab817f266c Mon Sep 17 00:00:00 2001
From: Dominik Vogt <vogt@linux.vnet.ibm.com>
Date: Mon, 11 Jan 2016 13:26:04 +0100
Subject: [PATCH] S/390: PR/56194: Disable test g++.dg/init/const9

---
 gcc/testsuite/g++.dg/init/const9.C | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/testsuite/g++.dg/init/const9.C b/gcc/testsuite/g++.dg/init/const9.C
index d733e95..9745882 100644
--- a/gcc/testsuite/g++.dg/init/const9.C
+++ b/gcc/testsuite/g++.dg/init/const9.C
@@ -1,5 +1,5 @@
 // PR c++/55893
-// { dg-final { scan-assembler-not "rodata" { target { ! hppa*-*-* } } } }
+// { dg-final { scan-assembler-not "rodata" { target { ! "hppa*-*-* s390*-*-*" } } } }
 
 struct foo
 {
-- 
2.3.0

Re: [PATCH 0/2] S/390 test case fixes.

[Jeff Law]

On 01/11/2016 03:47 AM, Dominik Vogt wrote:
> On Mon, Jan 11, 2016 at 11:46:21AM +0100, Dominik Vogt wrote:
>> The following two patches fix some test cases for S/390.
>
> Patch 1:
>
>    gcc.dg/tree-ssa/20040204-1.c does not fail on S/390 at the moment.
The series looks fine to me.

jeff

Re: [PATCH 4/2] S/390 test case fixes.

[Dominik Vogt]

[-- Attachment #1: Type: text/plain, Size: 269 bytes --]

On Mon, Jan 11, 2016 at 11:46:21AM +0100, Dominik Vogt wrote:
> The following two patches fix some test cases for S/390.

Patch 4:

  Adapt the test pattern in bind_c_array_params_2.f90 to S/390
  assemply output.

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt
IBM Germany

[-- Attachment #2: 0001-ChangeLog --]
[-- Type: text/plain, Size: 96 bytes --]

gcc/testsuite/ChangeLog

	* gfortran.dg/bind_c_array_params_2.f90: Fix test patterns for S/390.

[-- Attachment #3: 0001-S-390-Fix-test-pattern-in-gfortran.dg-bind_c_array_p.patch --]
[-- Type: text/x-diff, Size: 1154 bytes --]

From 0c9eb783da06a97de97e73a8ed8c7b345510e0cc Mon Sep 17 00:00:00 2001
From: Dominik Vogt <vogt@linux.vnet.ibm.com>
Date: Mon, 11 Jan 2016 16:14:00 +0100
Subject: [PATCH] S/390: Fix test pattern in
 gfortran.dg/bind_c_array_params_2.f90.

---
 gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 b/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90
index c8eb50d..e749958 100644
--- a/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90
+++ b/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90
@@ -16,6 +16,7 @@ integer :: aa(4,4)
 call test(aa)
 end
 
-! { dg-final { scan-assembler-times "\[ \t\]\[$,_0-9\]*myBindC" 1 { target { ! { hppa*-*-* } } } } }
+! { dg-final { scan-assembler-times "\[ \t\]\[$,_0-9\]*myBindC" 1 { target { ! { hppa*-*-* s390*-*-* } } } } }
 ! { dg-final { scan-assembler-times "myBindC,%r2" 1 { target { hppa*-*-* } } } }
+! { dg-final { scan-assembler-times "brasl\t%r\[0-9\]*,myBindC" 1 { target { s390*-*-* } } } }
 ! { dg-final { scan-tree-dump-times "test \\\(&parm\\." 1 "original" } }
-- 
2.3.0

Re: [PATCH 0/2] S/390 test case fixes.

[Andreas Krebbel]

> gcc/testsuite/ChangeLog
> 
> 	* g++.dg/Wframe-address.C: Test requires -mbackchain for s390*-*-*
> 	* g++.dg/Wno-frame-address.C: Ditto.

Applied. Thanks!

-Andreas-

Re: [PATCH 0/2] S/390 test case fixes.

[Andreas Krebbel]

> gcc/testsuite/ChangeLog
> 
> 	* gcc.dg/tree-ssa/20040204-1.c: Do not xfail on S/390.

Applied. Thanks!

-Andreas-

Re: [PATCH 3/2] S/390 test case fixes.

[Andreas Krebbel]

> gcc/testsuite/ChangeLog
> 
> 	PR c++/56194
> 	* g++.dg/init/const9.C: Disable test on S/390.

Applied. Thanks!

-Andreas-

Re: [PATCH 4/2] S/390 test case fixes.

[Andreas Krebbel]

> gcc/testsuite/ChangeLog
> 
> 	* gfortran.dg/bind_c_array_params_2.f90: Fix test patterns for S/390.

Applied.  Thanks!

-Andreas-