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
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
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
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
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
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
> >
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
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
?
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
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
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
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
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.
> > >
> > >
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
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
> >
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+
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
The following two patches fix some test cases for S/390.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
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
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
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
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
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
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
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
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
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
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
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
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
check the information I've added to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60401
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
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
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
-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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
^_^ ^_^
--
Dominik Vogt
IBM Germany
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
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
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
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
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
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
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
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
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
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. */
&
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 @@
> >>>
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
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
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
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
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
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
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
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
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
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
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
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.
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
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
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
&
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
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
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_
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
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
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
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:
> >>
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 - 100 of 373 matches
Mail list logo