Re: [PATCH v3] S/390: Add splitter for "and" with complement.

2016-07-28 Thread Dominik Vogt
Version 4 of the patch. Activated the patterns als for -mesa, as discussed internally. Bootstrapped and regression testes on s390 and s390x biarch. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.md ("*andc_split", "*andc_split2&q

[PATCH] S/390: Fix insv-1.c test with -m31.

2016-08-18 Thread Dominik Vogt
The atteched patch fixes the S/390 test case insv-1.c with -m31. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * gcc.target/s390/insv-1.c: Fix test when running with -m31. >From 4021fe9c4703e29b9446a24584ebd998c591d7ed Mon Sep 17 00:00:00 2001 F

[PATCH] S/390: Improve result verification in test case vec-genmask-1.c.

2016-08-18 Thread Dominik Vogt
t;vgmf" instructions are used. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * gcc.target/s390/zvector/vec-genmask-1.c: Improve result verification. >From 6750201c2b594bbbdce9abb14a3e1944806c425d Mon Sep 17 00:00:00 2001 From: Dominik Vogt Date: T

Re: [PATCH v3] S/390: Add splitter for "and" with complement.

2016-08-18 Thread Dominik Vogt
Version 5 of the patch with the splitter for -O0/-O1 removed, as discussed internally. Regression tested on s390x biarch and s390. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.md ("*andc_split"): New splitter for and with compl

Re: PATCH RFA: Call default_file_start from s390_asm_file_start

2016-08-23 Thread Dominik Vogt
t/s390/nolrl-1.c: Don't match the filename. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany >From 1475ed9918918f4181c5f1e00a905c378ffe9d7a Mon Sep 17 00:00:00 2001 From: Dominik Vogt Date: Tue, 23 Aug 2016 19:57:35 +0100 Subject: [PATCH] S/390: Add missing call to default_file_start

Re: [PING 4, PATCH] Remove xfail from thread_local-order2.C.

2016-08-23 Thread Dominik Vogt
On Mon, Jun 20, 2016 at 02:41:21PM +0100, Dominik Vogt wrote: > Patch: > https://gcc.gnu.org/ml/gcc-patches/2016-04/msg01587.html > > On Wed, Jan 27, 2016 at 10:39:44AM +0100, Dominik Vogt wrote: > > g++.dg/tls/thread_local-order2.C no longer fail with Glibc-2.18 or > >

Re: [PATCH v3] Take known zero bits into account when checking extraction.

2016-05-19 Thread Dominik Vogt
On Mon, May 16, 2016 at 01:09:36PM -0600, Jeff Law wrote: > On 05/11/2016 02:52 AM, Dominik Vogt wrote: > >On Wed, May 11, 2016 at 10:40:11AM +0200, Bernd Schmidt wrote: > >That's what I mentioned somewhere during the discussion. The s390 > >backend just uses COSTS_N_I

Re: [PATCH v4] Take known zero bits into account when checking extraction.

2016-05-23 Thread Dominik Vogt
On Thu, May 19, 2016 at 11:14:37AM -0600, Jeff Law wrote: > On 05/19/2016 05:18 AM, Dominik Vogt wrote: > >On Mon, May 16, 2016 at 01:09:36PM -0600, Jeff Law wrote: > >>> On 05/11/2016 02:52 AM, Dominik Vogt wrote: > >>>> >On Wed, May 11, 2016 a

Re: [RFA] Minor cleanup to allocate_dynamic_stack_space

2016-05-23 Thread Dominik Vogt
? The first part of the change (removing the dead code) is fine. However, I suggest to leave "must_align" in the code for now because I have another patch in the queue that assigns a calculated value to must_align. For that I'd have to revert this part of your patch, so I think it's not worth the effort to remove it in the first place. See https://gcc.gnu.org/ml/gcc-patches/2016-05/msg00445.html Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [RFA] Minor cleanup to allocate_dynamic_stack_space

2016-05-25 Thread Dominik Vogt
ne might expect > because of the reindentation that needs to happen. So I've included > a -b diff variant which shows how little actually changed here. > > This should have no impact on any target. > > Bootstrapped and regression tested on x86_64 linux. Ok for the trunk? I've rebased my patch on yours and also removed EXTRA_ALIGN which is also constant. I'll send new versions of both patches later. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH 1/2][v3] Drop excess size used for run time allocated stack variables.

2016-05-25 Thread Dominik Vogt
On Tue, May 03, 2016 at 03:17:53PM +0100, Dominik Vogt wrote: > Version two of the patch including a test case. > > On Mon, May 02, 2016 at 09:10:25AM -0600, Jeff Law wrote: > > On 04/29/2016 04:12 PM, Dominik Vogt wrote: > > >The attached patch removes excess sta

Re: [PATCH 2/2][v3] Drop excess size used for run time allocated stack variables.

2016-05-25 Thread Dominik Vogt
On Wed, May 25, 2016 at 02:30:54PM +0100, Dominik Vogt wrote: > On Tue, May 03, 2016 at 03:17:53PM +0100, Dominik Vogt wrote: > > Version two of the patch including a test case. > > > > On Mon, May 02, 2016 at 09:10:25AM -0600, Jeff Law wrote: > > > On 04/29/201

Re: [PATCH 2/2][v3] Drop excess size used for run time allocated stack variables.

2016-05-25 Thread Dominik Vogt
On Wed, May 25, 2016 at 02:32:41PM +0100, Dominik Vogt wrote: > On Wed, May 25, 2016 at 02:30:54PM +0100, Dominik Vogt wrote: > > On Tue, May 03, 2016 at 03:17:53PM +0100, Dominik Vogt wrote: > > > Version two of the patch including a test case. > > > > > >

Re: [PATCH v2, PING 1] Allocate constant size dynamic stack space in the prologue

2016-06-20 Thread Dominik Vogt
On Fri, May 06, 2016 at 10:44:15AM +0100, Dominik Vogt wrote: > > diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c > > index 21f21c9..4d48afd 100644 > > --- a/gcc/cfgexpand.c > > +++ b/gcc/cfgexpand.c > ... > > @@ -1099,8 +1101,10 @@ expand_stack_va

Re: [PATCH 2/2][v3] Drop excess size used for run time allocated stack variables.

2016-06-20 Thread Dominik Vogt
On Wed, Jun 08, 2016 at 01:21:09PM +0200, Bernd Schmidt wrote: > On 05/25/2016 03:32 PM, Dominik Vogt wrote: > > * explow.c (round_push): Use know adjustment. > > (allocate_dynamic_stack_space): Pass known adjustment to round_push. > >gcc/testsuite/ChangeLog > >

Re: [PING 2, PATCH] Remove xfail from thread_local-order2.C.

2016-06-20 Thread Dominik Vogt
Patch: https://gcc.gnu.org/ml/gcc-patches/2016-04/msg01587.html On Wed, Jan 27, 2016 at 10:39:44AM +0100, Dominik Vogt wrote: > g++.dg/tls/thread_local-order2.C no longer fail with Glibc-2.18 or > newer since this commit: > > 2014-08-01 Zifei Tong > > * libsupc+

Re: [PATCH] Drop excess size used for run time allocated stack variables.

2016-06-21 Thread Dominik Vogt
size = 0; > > /* Align stack boundary. Only needed if we're calling another function > or using alloca. */ > if (!crtl->is_leaf || cfun->calls_alloca > || ix86_current_function_calls_tls_descriptor) > offset = ROUND_UP (offset, preferred_alignment); > > or here in rs6000: > info->parm_size= RS6000_ALIGN (crtl->outgoing_args_size, > TARGET_ALTIVEC ? 16 : 8); > > which could probably use the default definition of > STACK_DYNAMIC_OFFSET instead of this, if the outgoing_args_size was > rounded appropriately: > #define STACK_DYNAMIC_OFFSET(FUNDECL) \ > (RS6000_ALIGN (crtl->outgoing_args_size, \ > (TARGET_ALTIVEC || TARGET_VSX) ? 16 : 8) \ >+ (STACK_POINTER_OFFSET)) Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH] Fix up handling of REG_EH_REGION notes in LRA (PR middle-end/69838)

2016-02-19 Thread Dominik Vogt
thout Ada), > and Dominik has kindly tested this on s390x-linux > (presumably with Ada, but don't know for sure). Yes, all languages with the whole testsuite. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PING, PATCH] libitm: Introduce target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE.

2016-02-22 Thread Dominik Vogt
On Mon, Feb 01, 2016 at 02:18:48PM +0100, Dominik Vogt wrote: > The attached patch adds the a target specific attribute via the > new target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE to the > function begin_transaction(). S/390 uses this to set the > soft-float target attribute which

Re: [PING 2, PATCH] S/390: Reduce accuracy of bessel_6.f90.

2016-02-22 Thread Dominik Vogt
On Mon, Jan 11, 2016 at 03:40:56PM +0100, Dominik Vogt wrote: > Another patch reducing the accuracy required in the bessel_6 test. > gcc/testsuite/ChangeLog > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. > >From 70a35dd6f6bf906d8e5907667ad0f04f981a61ac Mon Se

Re: [PING 2, PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-02-22 Thread Dominik Vogt
On Mon, Jan 04, 2016 at 12:33:21PM +0100, Dominik Vogt wrote: > On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > > >The attached patch fixes C++-11 handling of "alignas(0)" which > > >should be ign

Re: [PATCH] Fix PR69760

2016-02-29 Thread Dominik Vogt
overflow behavior. > > * gcc.dg/torture/pr69760.c: New testcase. This causes a regression on s390x w/ reassoc_6.f. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69760 for details. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

[PATCH] S/390: Ada: Enable Stack_Check_Probes.

2016-03-01 Thread Dominik Vogt
The attached patch enables Stack_Check_Probes for Ada on S/390[x] and thereby fixes the acats test cases c52103x, c52104x and cb1010a. Bootstrapped and regression tested on s390x (-m31, -m64) and s390. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ada/ChangeLog * system

Re: [PING 3, PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-03-01 Thread Dominik Vogt
On Mon, Jan 04, 2016 at 12:33:21PM +0100, Dominik Vogt wrote: > On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > > >The attached patch fixes C++-11 handling of "alignas(0)" which > > >should be ign

Re: [PING 3, PATCH] S/390: Reduce accuracy of bessel_6.f90.

2016-03-01 Thread Dominik Vogt
On Mon, Jan 11, 2016 at 03:40:56PM +0100, Dominik Vogt wrote: > Another patch reducing the accuracy required in the bessel_6 test. > gcc/testsuite/ChangeLog > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. > >From 70a35dd6f6bf906d8e5907667ad0f04f981a61ac Mon Se

Re: [PING 2, PATCH] libitm: Introduce target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE.

2016-03-01 Thread Dominik Vogt
On Mon, Feb 01, 2016 at 02:18:48PM +0100, Dominik Vogt wrote: > The attached patch adds the a target specific attribute via the > new target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE to the > function begin_transaction(). S/390 uses this to set the > soft-float target attribute which

Re: [PATCH] Additional tests for 69987, 69989, 69740

2016-03-02 Thread Dominik Vogt
f90 -O (test for excess errors) Excess errors: f951: Warning: -fprefetch-loop-arrays not supported for this target (try -march switches) Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

[PATCH] S/390: Set GOARCH to the current target when testing multiarch.

2016-03-02 Thread Dominik Vogt
The attached patch fixes a test failure of go.test/test/env.go on s390x biarch. Bootstrapped and regression tested on s390x biarch and s390. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69766 Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * go.test/go

Re: [PING 4, PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-03-14 Thread Dominik Vogt
On Mon, Jan 04, 2016 at 12:33:21PM +0100, Dominik Vogt wrote: > On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > > >The attached patch fixes C++-11 handling of "alignas(0)" which > > >should be ign

Re: [PING 5, PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-04-05 Thread Dominik Vogt
On Mon, Jan 04, 2016 at 12:33:21PM +0100, Dominik Vogt wrote: > On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > > >The attached patch fixes C++-11 handling of "alignas(0)" which > > >should be ign

Re: [PING 6, PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-04-12 Thread Dominik Vogt
On Mon, Jan 04, 2016 at 12:33:21PM +0100, Dominik Vogt wrote: > On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > > >The attached patch fixes C++-11 handling of "alignas(0)" which > > >should be ign

[PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2015-12-31 Thread Dominik Vogt
nu.org/bugzilla/show_bug.cgi?id=69089 Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/cp/ChangeLog * parser.c (cp_parser_std_attribute_spec): Do not generate an "aligned" attribute for "alignas(0)". gcc/testsuite/ChangeLog PR tree-optimization/6806

Re: [PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-01-04 Thread Dominik Vogt
On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > >The attached patch fixes C++-11 handling of "alignas(0)" which > >should be ignored but currently generates an error message. A > >test case is include

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

2016-01-11 Thread 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.

2016-01-11 Thread Dominik Vogt
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 gcc/testsuite/ChangeLog * gcc

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

2016-01-11 Thread Dominik Vogt
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 gcc/testsuite/ChangeLog

[PATCH] Fix line number that is expected to generate an error.

2016-01-11 Thread Dominik Vogt
The attached patch fixes a test failure caused by expecting the error message for the wrong line. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * g++.dg/cpp0x/constexpr-reinterpret1.C: Fix line number that is expected to generate an error. >F

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

2016-01-11 Thread Dominik Vogt
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 Germ

[PATCH] S/390: Better help text for -mmvcle.

2016-01-11 Thread Dominik Vogt
The attached patch improves the help text of the -mmvcle option on S/390. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.opt (mmvcle): More verbose help text. >From 817358341388950641967d709ca2945bc8305998 Mon Sep 17 00:00:00 2001 From: Domi

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

2016-01-11 Thread Dominik Vogt
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: > http

[PATCH] S/390: Reduce accuracy of bessel_6.f90.

2016-01-11 Thread Dominik Vogt
Another patch reducing the accuracy required in the bessel_6 test. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. >From 70a35dd6f6bf906d8e5907667ad0f04f981a61ac Mon Sep 17 00:00:00 2001 From: Domi

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

2016-01-14 Thread Dominik Vogt
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 gcc/testsuite/Change

[PATCH] S/390: Add missin full stop in option description.

2016-01-14 Thread Dominik Vogt
The attached patch fixes a test failure because of a missing full stop at the end of an S/390 option help text. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.opt (mtune): Add full stop. >From 6d4de7bb9ff9643b4cf3c967ea5ccfd1591d0799 Mon Sep 17

Re: [PING][PATCH] Fix line number that is expected to generate an error.

2016-01-18 Thread Dominik Vogt
On Mon, Jan 11, 2016 at 12:28:40PM +0100, Dominik Vogt wrote: > The attached patch fixes a test failure caused by expecting the > error message for the wrong line. Can this be committed? > gcc/testsuite/ChangeLog > > * g++.dg/cpp0x/constexpr-reinterpret1.C: Fix line

Re: [PATCH] S/390: Reduce accuracy of bessel_6.f90.

2016-01-18 Thread Dominik Vogt
On Mon, Jan 11, 2016 at 03:40:56PM +0100, Dominik Vogt wrote: > Another patch reducing the accuracy required in the bessel_6 test. Can this be committed? > gcc/testsuite/ChangeLog > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. > >From 70a35dd6f6bf906d8e5907

Re: [PATCH 2/5] s390: Fix missing .size directives.

2016-01-20 Thread Dominik Vogt
es to > the original macro > would not require adjusting our backend. Maybe it's better not to invent yet another solution to deal with this and just do it like proposed in the patch. So if the default implementation is ever changed, the same search pattern will find all identical copies of the code. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [patch] libstdc++/14608 Add C++-conforming wrappers for stdlib.h and math.h

2016-01-20 Thread Dominik Vogt
check the information I've added to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60401 Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH 4/4] Un-XFAIL ssa-dom-cse-2.c for most platforms

2016-01-26 Thread Dominik Vogt
able. */ > > -/* { dg-final { scan-tree-dump "return 28;" "optimized" { xfail aarch64*-*-* > alpha*-*-* hppa*-*-* powerpc*-*-* sparc*-*-* s390*-*-* } } } */ > +/* { dg-final { scan-tree-dump "return 28;" "optimized" { xfail alpha*-*-* > powerpc64*-*-* } } } */ > -- > 1.9.1 Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH] Fix PR69336

2016-01-26 Thread Dominik Vogt
rray for the nested braced init. > + */ > + constexpr std::pair map_data[] = { > + { 5, "apple" }, > + { 8, "pear" }, > + { 0, "banana" } > + }; > + > + template constexpr int cstrcmp(const char *a, const char *b) > + { > + for (size_t n = 0; n < N; n++) > + { > + if (a[n] < b[n]) return -1; > + if (a[n] > b[n]) return 1; > + } > + return 0; > + } > + > + int main(void) > + { > + constexpr auto cmap = make_static_map(map_data); > + // No abort() appears in assembler, so this was executed constexpr > + if(!cmap[8]) abort(); > + // This however does cause code implementing a lookup to be generated, > + // so this was NOT executed constexpr > + //const char *foo=cmap[5]; > + return 0; > + } > + > + // { dg-final { scan-tree-dump-not "cmap" "optimized" } } Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

[PATCH] Remove xfail from thread_local-order2.C.

2016-01-27 Thread Dominik Vogt
-0d04-0410-96 https://gcc.gnu.org/ml/gcc-patches/2014-07/msg02091.html So, is it time to remove the xfail from the test case? Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * g++.dg/tls/thread_local-order2.C: Remove xfail. >F

[PATCH] S/390: Require a hardware vector support for test to succeed.

2016-01-27 Thread Dominik Vogt
roach? Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * gcc.dg/tree-ssa/ssa-dom-cse-2.c: Require a hardware vector support for test to succeed. >From d855b255132d6a3ef46571e84aa75c95d8c3c737 Mon Sep 17 00:00:00 2001 From: Dominik Vogt Date: Wed, 27

[PATCH] PR other/69006: S/390: Fix extra newlines after diagnostics.

2016-01-27 Thread Dominik Vogt
The attached patch removes a blank line after an error message. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69006 Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog PR other/69006 * config/s390/s390-c.c (s390_resolve_overloaded_builtin): Remove trailing

[PATCH] PR c++/69462: Provide FLT_EVAL_METHOD and DECIMAL_DIG in float.h.

2016-01-28 Thread Dominik Vogt
The attached patch (written by Jonathan, not me) makes FLT_EVAL_METHOD and DECIMAL_DIG available in C++-11 as they should be. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69462 Can this be committed (should it wait for stage1)? Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog

Re: [PATCH] PR c++/69462: Provide FLT_EVAL_METHOD and DECIMAL_DIG in float.h.

2016-01-29 Thread Dominik Vogt
On Thu, Jan 28, 2016 at 03:41:29PM +0100, Jakub Jelinek wrote: > On Thu, Jan 28, 2016 at 01:40:12PM +0100, Dominik Vogt wrote: > > -#if defined (__STDC_VERSION__) && __STDC_VERSION__ >= 199901L > > +#if (defined (__STDC_VERSION__) && __STDC_VERSION__ >= 199901

Re: [PATCH] PR other/69006: S/390: Fix extra newlines after diagnostics.

2016-01-29 Thread Dominik Vogt
On Wed, Jan 27, 2016 at 09:22:19AM -0500, David Malcolm wrote: > On Wed, 2016-01-27 at 12:01 +0100, Dominik Vogt wrote: > > The attached patch removes a blank line after an error message. > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69006 > > Presumably this was ex

[PATCH] S/390: Use %qs in error messages.

2016-01-29 Thread Dominik Vogt
The attached patch replaces %qs instead of %s in an error message, adds %qs to another and fixes indentation in one of the messages. Compiled and checked that no tests rely on the changed error messages on a zEC12. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog

Re: [PING, PATCH] Reduce accuracy of bessel_6.f90.

2016-01-29 Thread Dominik Vogt
On Mon, Jan 11, 2016 at 03:40:56PM +0100, Dominik Vogt wrote: > Another patch reducing the accuracy required in the bessel_6 test. Fixes the test case for S/390. Can this be committed? > gcc/testsuite/ChangeLog > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/3

[PATCH] S/390: Correct documentation typos.

2016-02-01 Thread Dominik Vogt
The attached patch corrects a few typos in the S/390 specific documentation. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * doc/extend.texi: S/390: Correct some typos. >From 418db8fbac09ecfb8f70f7f2a5c8e4ce52cc160a Mon Sep 17 00:00:00 2001 From: Dominik Vogt D

[PATCH] libitm: Introduce target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE.

2016-02-01 Thread Dominik Vogt
document internal macros like USE_HTM_FASTPATH or the new macro, I've put the documentation in a comment where the macro is used. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany libitm/ChangeLog * config/s390/target.h (TARGET_BEGIN_TRANSACTION_ATTRIBUTE): Define fun

[PATCH] libstdc++: S/390: Add missing baseline_symbols.txt for s390x/-m31.

2016-02-01 Thread Dominik Vogt
The attached patch copies the existing libstdc++-v3/config/abi/post/s390-linux-gnu/baseline_symbols.txt to .../s390x-linux-gnu/32/baseline_symbols.txt. This fixes the abi test failure on s390x with -m31. The patch is gzip'ed because it's large. Ciao Dominik ^_^ ^_^ -- Domini

Re: [PATCH 4/4] Un-XFAIL ssa-dom-cse-2.c for most platforms

2016-02-04 Thread Dominik Vogt
On Wed, Feb 03, 2016 at 11:41:02AM +, Alan Lawrence wrote: > On 26/01/16 12:23, Dominik Vogt wrote: > >On Mon, Dec 21, 2015 at 01:13:28PM +, Alan Lawrence wrote: > >>...the test passes with --param sra-max-scalarization-size-Ospeed. > >> > >>Verified on

[PATCH] S/390: Do not require -march=z13 on s390 but only on s390x.

2016-02-04 Thread Dominik Vogt
A previous patch required -march=z13 in ssa-dom-cse-2.c for s390 and s390x, but actually only s390x really needs it. The attached patch removes the extra option on s390 (and fixes comment formatting). Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * gcc.dg

[PATCH] S/390: PR 69625: Add test case

2016-02-05 Thread Dominik Vogt
The attached patch adds a testcase for PR 69625. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany ChangeLog * gcc.target/s390/pr69625.c: Add test case. >From 08df803a901078cabca938d0a7d0d6120b7c6132 Mon Sep 17 00:00:00 2001 From: Dominik Vogt Date: Fri, 5 Feb 2016 15:13:08 +0

Re: [PING, PATCH] PR/68089: C++-11: Ingore "alignas(0)".

2016-02-05 Thread Dominik Vogt
Can this be approved? https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69089 On Mon, Jan 04, 2016 at 12:33:21PM +0100, Dominik Vogt wrote: > On Fri, Jan 01, 2016 at 05:53:08PM -0700, Martin Sebor wrote: > > On 12/31/2015 04:50 AM, Dominik Vogt wrote: > > >The attached patch fix

Re: [PATCH] S/390: PR 69625: Add test case

2016-02-09 Thread Dominik Vogt
On Fri, Feb 05, 2016 at 05:07:57PM +0100, Dominik Vogt wrote: > The attached patch adds a testcase for PR 69625. Version 2 also runs with -m31. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany ChangeLog * gcc.target/s390/pr69625.c: Add test case. &g

[PATCH 0/3] S/390: Improved risbg usage.

2016-09-20 Thread Dominik Vogt
^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH 1/3] S/390: Mode attrs "bitoff[_plus]" simplify risbg instructions.

2016-09-20 Thread Dominik Vogt
On Tue, Sep 20, 2016 at 01:37:18PM +0100, Dominik Vogt wrote: > The following series of patches improves usage of the risbg and > risbgn instructions on s390/s390x. The patches have been > regression tested on s390 and s390x and pass the Spec2006 > testsuite without any nega

Re: [PATCH 2/3] S/390: Enable wraparound in s390_contiguous_bitmask_p.

2016-09-20 Thread Dominik Vogt
On Tue, Sep 20, 2016 at 01:37:18PM +0100, Dominik Vogt wrote: > The following series of patches improves usage of the risbg and > risbgn instructions on s390/s390x. The patches have been > regression tested on s390 and s390x and pass the Spec2006 > testsuite without any negative effec

Re: [PATCH 3/3] S/390: Improved risbg usage.

2016-09-20 Thread Dominik Vogt
On Tue, Sep 20, 2016 at 01:37:18PM +0100, Dominik Vogt wrote: > The following series of patches improves usage of the risbg and > risbgn instructions on s390/s390x. The patches have been > regression tested on s390 and s390x and pass the Spec2006 > testsuite without any negative effec

Re: [PATCH] Extend dg-{error,warning,message,bogus} line specification to allow relative line numbers

2016-09-23 Thread Dominik Vogt
So, the test expects an error: /* { dg-error "arguments to .-mhotpatch=n,m. should be non-negative integers" "" { target *-*-* } 1 } */ but no longer matches the error that really occurs? Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH] Extend dg-{error,warning,message,bogus} line specification to allow relative line numbers

2016-09-23 Thread Dominik Vogt
On Fri, Sep 23, 2016 at 04:48:36PM +0200, Jakub Jelinek wrote: > On Fri, Sep 23, 2016 at 03:43:11PM +0100, Dominik Vogt wrote: > > Great! > > > > But could this patch be responsible with some dg-error related > > test errors on s390x that are present with current HEA

[PATCH]: S/390: Fix hotpatch test cases.

2016-09-23 Thread Dominik Vogt
THe attached patch fixes some dg-error tests that were broken since the recent change of the error location. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * gcc.target/s390/hotpatch-compile-1.c: Fixed dg-error test. * gcc.target/s390/hotpatch

[PATCH] Allocate constant size dynamic stack space in the prologue

2015-11-25 Thread Dominik Vogt
ze is done completely in the middleend, I don't see a way to fix this in the backend. The patch has been tested (only) on S/390[x]. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * cfgexpand.c (expand_stack_vars): Implement ALLOCATE_DYNAMIC_STACK_SPAC

Re: [PATCH] Allocate constant size dynamic stack space in the prologue

2015-11-25 Thread Dominik Vogt
On Wed, Nov 25, 2015 at 02:31:38PM +0100, Bernd Schmidt wrote: > On 11/25/2015 01:56 PM, Dominik Vogt wrote: > >The attached patch fixes a warning during Linux kernel compilation > >on S/390 due to -mwarn-dynamicstack and runtime alignment of stack > >variables with const

Re: [PATCH] Allocate constant size dynamic stack space in the prologue

2015-11-27 Thread Dominik Vogt
calculations are the same for both. * Use a target hook to activate the feature (for now). (This is just meant to make it more feasible to be included in Gcc6. If it's to late for this the code may be as well be used for all targets.) Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM German

Re: [PATCH] Allocate constant size dynamic stack space in the prologue

2015-11-27 Thread Dominik Vogt
On Fri, Nov 27, 2015 at 03:09:15PM +0100, Dominik Vogt wrote: > +++ b/gcc/config/s390/s390.h ... > +/* Constant size dynamic stack space can be allocated through the function > + prologue to save the extra instructions to adjust the stack pointer. */ &

Re: [PATCH] S/390: Fix warning in "*movstr" pattern.

2015-11-30 Thread Dominik Vogt
On Mon, Nov 09, 2015 at 01:33:23PM +0100, Andreas Krebbel wrote: > On 11/04/2015 02:39 AM, Dominik Vogt wrote: > > On Tue, Nov 03, 2015 at 06:47:28PM +0100, Ulrich Weigand wrote: > >> Dominik Vogt wrote: > >> > >>> @@ -2936,7 +2936,7 @@ > >>>

S/390: Fix warnings in "*setmem_long..." patterns.

2015-11-30 Thread Dominik Vogt
ithout any measurable changes in performance. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.c (s390_expand_setmem): Use new expanders. * config/s390/s390.md ("*setmem_long") ("*setmem_long_and", &qu

Re: S/390: Fix warnings in "*setmem_long..." patterns.

2015-12-01 Thread Dominik Vogt
On Mon, Nov 30, 2015 at 06:11:33PM +0100, Ulrich Weigand wrote: > On 11/30/2015 04:11 PM, Dominik Vogt wrote: > > The attached patch fixes some warnings generated by the setmem... > > patterns in s390.md during build and add test cases for the > > patterns. The patch is to be

Re: S/390: Fix warnings in "*setmem_long..." patterns.

2015-12-01 Thread Dominik Vogt
On Tue, Dec 01, 2015 at 10:59:54AM +0100, Dominik Vogt wrote: > @@ -3336,11 +3342,12 @@ > (set_attr "type" "vs")]) > > (define_insn "*setmem_long_31z" > - [(clobber (match_operand:TI 0 "register_operand" "=d")) > - (s

Re: [PATCH] S/390: Fix warning in "*movstr" pattern.

2015-12-02 Thread Dominik Vogt
On Wed, Dec 02, 2015 at 09:59:10AM +0100, Andreas Krebbel wrote: > On 11/30/2015 03:45 PM, Dominik Vogt wrote: > > On Mon, Nov 09, 2015 at 01:33:23PM +0100, Andreas Krebbel wrote: > >> On 11/04/2015 02:39 AM, Dominik Vogt wrote: > >>> On Tue, Nov 03, 2015 at 06:47:28

Re: [PATCH] S/390: Fix warning in "*movstr" pattern.

2015-12-02 Thread Dominik Vogt
On Wed, Dec 02, 2015 at 10:11:54AM +0100, Dominik Vogt wrote: > On Wed, Dec 02, 2015 at 09:59:10AM +0100, Andreas Krebbel wrote: > > On 11/30/2015 03:45 PM, Dominik Vogt wrote: > > > On Mon, Nov 09, 2015 at 01:33:23PM +0100, Andreas Krebbel wrote: > > >> On 11/04/201

Re: S/390: Fix warnings in "*setmem_long..." patterns.

2015-12-02 Thread Dominik Vogt
dded. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.c (s390_expand_setmem): Use new expanders. * config/s390/s390.md ("*setmem_long") ("*setmem_long_and", "*setmem_long_31z"): Fix warnings. ("*setmem_lon

Re: [PATCH] S/390: Fix warning in "*movstr" pattern.

2015-12-03 Thread Dominik Vogt
Version 5 with two fixes to the test case. :-/ (ChangeLog is the same.) Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany >From 5965f62501b271285bacb90b11ef3f748338d1cf Mon Sep 17 00:00:00 2001 From: Dominik Vogt Date: Tue, 3 Nov 2015 18:03:02 +0100 Subject: [PATCH] S/390: Fix warning

[PATCH] S/390: Add -mbackchain options to fix test failure.

2015-12-03 Thread Dominik Vogt
On S/390, __builtin_return_address and __builtin_frame_address require the -mbackchain option to work for arbitrary stack frames. The attached patch adds the option to two test cases to make them work. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * gcc.dg

Re: [PR64164] drop copyrename, integrate into expand

2015-12-04 Thread Dominik Vogt
t; > Regstrapped on x86_64-linux-gnu native and on i686-pc-linux-gnu native > on x86_64, so without lto. Is this ok to install? The patch that got committed as a result of this discussion causes a performance regression on s390[x]. Bug report: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68695 Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH 1/2] s/390: Implement "target" attribute.

2015-12-04 Thread Dominik Vogt
Next version of the patch with the changes discussed internally. (Sorry for the version number; I now use my internam branch numbers for sake of clarity.) Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.opt (s390_arch_string): Remove

Re: [PATCH] S/390: Fix warning in "*movstr" pattern.

2015-12-04 Thread Dominik Vogt
Version 6 with another fix. This should work now. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.md ("movstr", "*movstr"): Fix warning. ("movstr"): New indirect expanders used by "movstr". gcc/testsu

Re: S/390: Fix warnings in "*setmem_long..." patterns.

2015-12-04 Thread Dominik Vogt
Version 5 with the latest requested changes. Seems to work now. I've dropped the extra patch and rather marked the failing tests as "xfail". Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.c (s390_expand_setmem): Use new expanders.

[PATCH] S/390: Wide int support.

2015-12-11 Thread Dominik Vogt
The attached patch introduces wide int support to S/390 in order to resolve a test case failure in gcc.dg/pr68129_1.c that is caused by an assertion in simplify-rtx.c:simplify_const_binary_operation(). Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390

[PATCH] S/390: Allow to use r1 to r4 as literal pool base.

2015-12-11 Thread Dominik Vogt
The attached patch enables using r1 to r4 as the literal pool base pointer if one of them is unused in a leaf function. The unpatched code supports only r5 and r13. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.c (s390_init_frame_layout): Allow

Re: [PATCH] S/390: Allow to use r1 to r4 as literal pool base.

2015-12-15 Thread Dominik Vogt
On Mon, Dec 14, 2015 at 04:08:32PM +0100, Ulrich Weigand wrote: > Dominik Vogt wrote: > > > The attached patch enables using r1 to r4 as the literal pool base pointer > > if > > one of them is unused in a leaf function. The unpatched code supports only > > r5 &

Re: ipa-cp heuristics fixes

2015-12-16 Thread Dominik Vogt
AIL: gcc.dg/guality/vla-1.c -O3 -g line 17 i == 5 FAIL: gcc.dg/guality/vla-1.c -O3 -g line 17 sizeof (a) == 6 FAIL: gcc.dg/guality/vla-1.c -Os line 17 sizeof (a) == 6 What can I do to help fixing this? Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: ipa-cp heuristics fixes

2015-12-16 Thread Dominik Vogt
On Wed, Dec 16, 2015 at 10:59:10AM +0100, Richard Biener wrote: > Same on x86_64 btw. If there isn't a bugreport already please open > one to track this issue. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68935 Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

[RFC][PATCH] Fix broken handling of LABEL_REF in genrecog + genpreds.

2015-12-16 Thread Dominik Vogt
necessary to add "const_int" (or similar) to match_code of some predicates handling label_ref but not const_int. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/predicates.md ("larl_operand"): Remove now superfluous const_

Re: [PATCH] S/390: Allow to use r1 to r4 as literal pool base.

2015-12-16 Thread Dominik Vogt
On Wed, Dec 16, 2015 at 01:51:45PM +0100, Ulrich Weigand wrote: > Dominik Vogt wrote: > > > r2 through r4 should be fine. [ Not sure if there will be many (any?) > > > cases > > > where one of those is unused but r5 isn't, however. ] > > > > Thi

Re: [PATCH] S/390: Allow to use r2 to r4 as literal pool base.

2015-12-18 Thread Dominik Vogt
Latest patch for r2 to r4, including a test case. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.c (s390_init_frame_layout): Try r4 to r2 for the literal pool pointer. gcc/testsuite/ChangeLog * gcc.target/s390/litpool-r3-1.c: New

Re: [RFC][PATCH] Fix broken handling of LABEL_REF in genrecog + genpreds.

2015-12-22 Thread Dominik Vogt
On Fri, Dec 18, 2015 at 09:55:54AM +, Richard Sandiford wrote: > Dominik Vogt writes: > > The attached patch fixes the handling of LABEL_REF in genrecog and > > genpreds. > > > > The current code assumes that X can have only a mode than PRED (X, > > MODE) i

Re: [RFC] Check number of uses in simplify_cond_using_ranges().

2016-11-08 Thread Dominik Vogt
On Fri, Nov 04, 2016 at 01:54:20PM +0100, Richard Biener wrote: > On Fri, Nov 4, 2016 at 12:08 PM, Dominik Vogt wrote: > > On Fri, Nov 04, 2016 at 09:47:26AM +0100, Richard Biener wrote: > >> On Thu, Nov 3, 2016 at 4:03 PM, Dominik Vogt > >> wrote: > >>

[PATCH] S390: Fix PR/77822.

2016-11-08 Thread Dominik Vogt
The attached patch fixes PR/77822 on s390/s390x dor gcc-6 *only*. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77822 Bootstrapped and regression tested on s390 and s390x biarch on a zEC12. For gcc-7, there will be a different patch. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc

  1   2   3   4   >