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
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
The tests validate the syntax of the options an the
attribute and check for the correct number of nops. I have
verified the correct layout of the trampoline manually (i.e. the
correct placement of the nops).
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
2013-12-04 Dominik Vogt
Backporting this patch to 4.8, 4.4 and 4.1 brougth some issues to
light. The new version of the patch optimizes some code and
warnings and adapts the test cases so that they run without
modifications on 4.8 and earlier.
Original message:
On Thu, Dec 05, 2013 at 09:06:31AM +0100, Dominik Vogt
in the code since z9-ec support was added.
* 0003 -march=native now detects various cpu features and passes
that as options from the driver to the compiler.
* 0004 Non-mandatory cleanup of the S390 code handling the various
cpu types.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0001 Fix cross compilation for S390 targets that was broken with
>the original -march=native aptch.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0002 Fix handling of -march=z9_ec in old code. This bug has been
>in the code since z9-ec support was added.
Ciao
Dominik ^_^ ^_^
--
D
On Fri, Jul 17, 2015 at 05:00:50PM +0100, Dominik Vogt wrote:
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0003 -march=native now detects various cpu features and passes
>that as options
On Fri, Jul 17, 2015 at 05:00:50PM +0100, Dominik Vogt wrote:
> The following series of patches contains fixes, and cleanup work
> related to -march=native (and -mtune=native) for S390.
>
> * 0004 Non-mandatory cleanup of the S390 code handling the various
>cpu types.
ry, some old code sneaked into the patch. Updated version
without the dead code attached. The ChangeLog ist still the same.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
>From b7942651a85761dcd9cf1f1c0c01074b9c327e43 Mon Sep 17 00:00:00 2001
From: Dominik Vogt
Date: Mon, 6 Jul 2015 1
On Sat, Jul 18, 2015 at 01:09:26AM +0200, Ulrich Weigand wrote:
> Dominik Vogt wrote:
>
> > + opt_esa_zarch = (has_highgprs) ? " -mzarch" : " -mesa";
>
> This will force -mesa on old machines *even in -m64 mode*,
> which is wrong and will cause comp
On Tue, Jul 21, 2015 at 01:14:44PM +0200, Ulrich Weigand wrote:
> Dominik Vogt wrote:
> This version is looking good, except for one problem:
>
> > - "%{!m31:%{!m64:-m64}}", \
> > - "%{!mesa
On Tue, Jul 21, 2015 at 07:05:27PM +0200, Ulrich Weigand wrote:
> Dominik Vogt wrote:
> > * config/s390/driver-native.c (s390_host_detect_local_cpu): Handle
> > processor capabilities with -march=native.
> > * config/s390/s390.h (MARCH_MTUNE_NA
etter for consistency and sanity do what other
> targets do, even if it won't break the cross-compilation.
> Do you agree?
It's certainly better to not compile a file with code that was
written to run on a different platform. I'll make a patch in a
couple of days.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
arlier post. (Andreas
will commit them shortly.)
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
uot;native" is one
of the allowed values.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
problem at the
moment.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* config.hosto_plugin_soname): Include driver-native.c only when
building with s390* as host and target.
>From fdf3a1f9ccffdbf89e9b262e657ac77765a6be1f Mon Sep 17 00:00:00 2001
From: Dominik V
or: unrecognized argument in option ‘-march=native’
cc1: note: valid arguments to ‘-march=’ are: g5 g6 native z10 z13 z196 z9-109
z9-ec z900 z990 zEC12
The patched code prints DriverOnly enum values only when the
driver generates the error message.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM German
ng it after the switch, but
it's not a big deal to change that.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* defaults.h (ASM_OUTPUT_START_FUNCTION_HEADER): New hook function.
(ASM_OUTPUT_START_FUNCTION_FOOTER): Ditto.
* varasm.c (assemble_start
e dying input pseudo.
This check-in caused a regression on s390, please see
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61578 for details.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
anges to
hooks.[ch] could be dropped.)
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* target.def: Add function_start and function_end hooks.
* hooks.c (hook_void_FILEptr_tree): New function.
* hooks.h: Ditto.
* varasm.c (assemble_start_functio
On Thu, Sep 24, 2015 at 12:41:24PM +0200, Bernd Schmidt wrote:
> On 09/23/2015 04:48 PM, Dominik Vogt wrote:
> >On Tue, Sep 22, 2015 at 01:56:15PM -0600, Jeff Law wrote: > Is
> >there some good reason these aren't hooks?
> >
> >No, that was just inobservance.
The old way ist still supported but may be removed once the
".machine" feature is supported by all as versions in the field.
The second patch depends on the first one, and both require the
(latest) change proposed in this thread:
https://gcc.gnu.org/ml/gcc-patches/2015-09/msg01546.h
On Fri, Sep 25, 2015 at 02:59:41PM +0100, Dominik Vogt wrote:
> The following set of two patches implements the function
> __attribute__ ((target("..."))) and the corresponding #pragma GCC
> target("...") on S/390. It comes with certain limitations:
>
>
On Fri, Sep 25, 2015 at 02:59:41PM +0100, Dominik Vogt wrote:
> The following set of two patches implements the function
> __attribute__ ((target("..."))) and the corresponding #pragma GCC
> target("...") on S/390. It comes with certain limitations:
>
>
On Fri, Sep 25, 2015 at 03:33:56PM +0200, Bernd Schmidt wrote:
> On 09/24/2015 03:48 PM, Dominik Vogt wrote:
> >Hm, I wonder whether wrapping all these section switches in
> >assemble_start/end_function in ".machine" pseudoops (that's what
> >we need the hooks
On Mon, Sep 28, 2015 at 01:54:17PM +0200, Bernd Schmidt wrote:
> On 09/28/2015 11:44 AM, Dominik Vogt wrote:
> >I mean, it's more or less random whether switching to and from the
> >function's section ends up inside the new .machine and
> >.machinemode directi
pop_options): Do not call
default_target_option_pragma_parse to prevent its warning when using
"#pragma GCC pop_options" on platforms that do not support
"#pragma GCC target".
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
>From d149dd8b9d6c9f720809de3839f2ad5
On Tue, Oct 13, 2015 at 02:28:37PM +0200, Bernd Schmidt wrote:
> On 10/13/2015 02:02 PM, Dominik Vogt wrote:
> >When "#pragma GCC pop_options" is used on a platform without
> >support for "#pragma GCC target", Gcc emits a warning. As
> >pop_options is us
hed.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* targhooks.c (default_target_option_pragma_parse): Do not warn if
called on behalf of "#pragma GCC pop_options".
gcc/testsuite/ChangeLog
* gcc.dg/pragma-pop_options-1.c: New test.
>From 4bb0068875e005b2f0e
On Mon, Sep 21, 2015 at 12:31:58PM +0100, Dominik Vogt wrote:
> This patch adds to new backend hooks
> ASM_OUTPUT_START_FUNCTION_HEADER and
> ASM_OUTPUT_END_FUNCTION_FOOTER that may be defined to emit
> assembly code at the very start or end of a function.
We no longer need this
On Fri, Sep 25, 2015 at 02:59:41PM +0100, Dominik Vogt wrote:
> The following set of two patches implements the function
> __attribute__ ((target("..."))) and the corresponding #pragma GCC
> target("...") on S/390. It comes with certain limitations:
>
>
I'll keep trying, but I'm pessimistic.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
On Fri, Sep 25, 2015 at 02:59:41PM +0100, Dominik Vogt wrote:
> The following set of two patches implements the function
> __attribute__ ((target("..."))) and the corresponding #pragma GCC
> target("...") on S/390. It comes with certain limitations:
>
>
On Fri, Sep 25, 2015 at 02:59:41PM +0100, Dominik Vogt wrote:
> The following set of two patches implements the function
> __attribute__ ((target("..."))) and the corresponding #pragma GCC
> target("...") on S/390. It comes with certain limitations:
>
>
/* Cast to int to avoid Warning "comparison is always true". */
> What cast?
Sorry, just forgot to remove the comment when changing the code
last time.
> - || new_opts_set.x_s390_warn_dynamicstack_p
> - )
> + || new_opts_set.x_s390_warn_dynamicstack_p)
O
new_opts_set.x_s390_tune)
joined_opts_set->x_s390_tune = new_opts_set->x_s390_tune;
(The opts_set parameter should really be const; I'll check if this
can be done.)
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
ree != new_tree)
> > +{
> > + cl_target_option_restore (&global_options, TREE_TARGET_OPTION
> > (new_tree));
> > + if (TREE_TARGET_GLOBALS (new_tree))
> > + restore_target_globals (TREE_TARGET_GLOBALS (new_tree));
> > + else if (new_tree == target_option_default_node)
> > + restore_target_globals (&default_target_globals);
> > + else
> > + TREE_TARGET_GLOBALS (new_tree) = save_target_globals_default_opts ();
>
> This last part is almost s390_reset_previous_fndecl. Perhaps it could
> be merged?!
Done.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
The attached patch fixes the annoying warnings generated by
diagnostic_set_last_function.
Can this be committed?
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* tree-diagnostic.h (diagnostic_set_last_function): Fix warning.
>F
On Mon, Nov 02, 2015 at 09:57:22AM -0700, Jeff Law wrote:
> On 11/02/2015 06:26 AM, Dominik Vogt wrote:
> >The attached patch fixes the annoying warnings generated by
> >diagnostic_set_last_function.
> Can you point out what warning you're fixing?
"TARGET_64BIT || !TARGET_ZARCH"
"mvcle\t%0,%1,%Y2\;jo\t.-4"
[(set_attr "length" "8")
(set_attr "type" "vs")])
("shift_count_or_setmem_operand" is the fill byte coded into the
instructions operand in the form
(Debug code removed from patch.)
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* genrecog.c (validate_pattern): Allow "set VOIDmode -> BLKmode" without
warnings.
>From 04376919c108c42a2e9835dd1809b198bc47513f Mon Sep 17 00:00:00 2001
From: Do
The attached patch fixes the message "warning: dest missing a
mode?" from s390.md.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* config/s390/s390.md ("*movstr"): Fix warning.
>From e03251cacf2004e4cb302de03b44bb1a3f6ad827 Mon Sep 17 00:0
On Tue, Nov 03, 2015 at 06:47:28PM +0100, Ulrich Weigand wrote:
> Dominik Vogt wrote:
>
> > @@ -2936,7 +2936,7 @@
> > (set (mem:BLK (match_operand:P 1 "register_operand" "0"))
> > (mem:BLK (match_operand:P 3 "register_operand" "2
The following series of patches fixes all occurences of
left-shifting negative constants in C code which is undefined by
the C standard. The patches have been tested on s390x, covering
only a small subset of the changes.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
On Tue, Nov 10, 2015 at 12:11:23PM +0100, Dominik Vogt wrote:
> The following series of patches fixes all occurences of
> left-shifting negative constants in C code which is undefined by
> the C standard. The patches have been tested on s390x, covering
> only a small subset of
Sorry, wrong mailing list, please ignore.
On Tue, Nov 10, 2015 at 12:11:23PM +0100, Dominik Vogt wrote:
> The following series of patches fixes all occurences of
> left-shifting negative constants in C code which is undefined by
> the C standard. The patches have been tested on s390x,
On Tue, Nov 10, 2015 at 12:11:23PM +0100, Dominik Vogt wrote:
> The following series of patches fixes all occurences of
> left-shifting negative constants in C code which is undefined by
> the C standard. The patches have been tested on s390x, covering
> only a small subset of
On Fri, Nov 06, 2015 at 03:16:37PM -0700, Jeff Law wrote:
> On 11/03/2015 02:57 AM, Dominik Vogt wrote:
> >On Mon, Nov 02, 2015 at 09:57:22AM -0700, Jeff Law wrote:
> >>On 11/02/2015 06:26 AM, Dominik Vogt wrote:
> >>>The attached patch fixes the
On Mon, Nov 09, 2015 at 02:09:31PM +0100, Andreas Krebbel wrote:
> On 11/02/2015 09:44 AM, Dominik Vogt wrote:
> > (@Uli: I'd like to hear your opinion on this issue.
> > Original message:
> > https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03403.html).
> >
> &
The attached patch fixes the S/390 patterns using the "pfpo"
instruction in s390.md. The instructions clobber r1, but the
patterns did not reflect that.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* config/s390/s390.md (GPR1_REGNUM): Ne
On Tue, Oct 13, 2015 at 11:06:48AM -0600, Jeff Law wrote:
> On 10/13/2015 07:12 AM, Dominik Vogt wrote:
> >In some cases, the work of the cse1 pass is counterproductive, as
> >we noticed on s390x. The effect described below is present since
> >at least 4.8.0. Note tha
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 fixes a bessel_7 fortran test failure that appeared during
SLES 12 distro tests for s390x - by bumping the error tolerance.
The same patch should apply to gcc-4.9 and gcc-4.8.
ChangeLog:
--
2014-08-25 Dominik Vogt
* gfortran.dg/bessel_7.f90: Bump allowed precision
On Fri, Oct 03, 2014 at 06:19:46PM -0700, Ian Lance Taylor wrote:
> On Tue, Sep 9, 2014 at 5:53 AM, Dominik Vogt wrote:
> > This patch fixes the compiler flags in libgo/mksysinfo.sh. In one
> > place, some compiler flags were missing that are consistently used
> > elswhere,
1 - 100 of 373 matches
Mail list logo
