[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

[PATCH] libitm: Introduce target macro TARGET_BEGIN_TRANSACTION_ATTRIBUTE.

2016-02-01 Thread Dominik Vogt
to 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 function

[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 ^_^ ^_^ -- Dominik Vogt IBM

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

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

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

[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

[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

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

2016-01-27 Thread Dominik Vogt
vn/gcc/trunk@213504 138bc75d-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.

[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] S/390: Require a hardware vector support for test to succeed.

2016-01-27 Thread Dominik Vogt
? 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 <v...@linux.vnet.i

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
which > + is less succinct than using a constexpr C array for the nested braced init. > + */ > + constexpr std::pair map_data[] = { > + { 5, "apple" }, > + { 8, "pear" }, > + { 0, "banana" } > + }; > + > + template constexpr int cstrc

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

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

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: [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:

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 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: [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

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

[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

[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] 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

[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] 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 incl

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

2015-12-31 Thread Dominik Vogt
g/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/68069

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 <v...@linux.vnet.ibm.com> writes: > > The attached patch fixes the handling of LABEL_REF in genrecog and > > genpreds. > > > > The current code assumes that X can have only a

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

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

2015-12-16 Thread Dominik Vogt
to this it is 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. ] > > > > This can

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

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 &

[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: [PR64164] drop copyrename, integrate into expand

2015-12-04 Thread Dominik Vogt
ve 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.

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 <v...@linux.vnet.ibm.com> Date: Tue, 3 Nov 2015 18:03:02 +0100 Subjec

[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: [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
o 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_long_and_64&q

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 i

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

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

2015-11-30 Thread Dominik Vogt
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", "*setmem

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 @@ > >>>

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] 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 Germany gcc

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

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

2015-11-25 Thread Dominik Vogt
size 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_SPACE_IN_PROLO

Re: [RFC] Cse reducing performance of register allocation with -O2

2015-11-23 Thread Dominik Vogt
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

[PATCH] S/390: Clobber r1 in patterns resulting in pfpo instruction.

2015-11-18 Thread Dominik Vogt
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

Re: [PATCH] Fix warning in tree-diagnostic.h.

2015-11-17 Thread Dominik Vogt
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 t

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

2015-11-17 Thread Dominik Vogt
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). > > > > O

[PATCH 0/2] Fix invalid left shift of negative value.

2015-11-10 Thread Dominik Vogt
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

Re: [PATCH 1/2] Fix invalid left shift of negative value.

2015-11-10 Thread Dominik Vogt
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

Re: [PATCH 2/2] Fix invalid left shift of negative value.

2015-11-10 Thread Dominik Vogt
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

Re: [PATCH 0/2] Fix invalid left shift of negative value.

2015-11-10 Thread Dominik Vogt
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,

Re: [PATCH] Fix warning in tree-diagnostic.h.

2015-11-03 Thread Dominik Vogt
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? Sur

[PATCH][RFC] Remove warning for SET VOIDmode -> BLKmode.

2015-11-03 Thread Dominik Vogt
uot;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 "

Re: [PATCH][RFC] Remove warning for SET VOIDmode -> BLKmode.

2015-11-03 Thread Dominik Vogt
(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:

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

2015-11-03 Thread Dominik Vogt
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

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

2015-11-03 Thread Dominik Vogt
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

[PATCH] Fix warning in tree-diagnostic.h.

2015-11-02 Thread Dominik Vogt
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

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

2015-11-02 Thread Dominik Vogt
_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

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

2015-11-02 Thread Dominik Vogt
ctions, but *actually* the code immediately modifies the global_opts structure when the pragma is parsed. I wonder if this is intentional. > > + if (old_tree != new_tree) > > +{ > > + cl_target_option_restore (_options, TREE_TARGET_OPTION > > (new_tree)); > > + if (

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

2015-10-31 Thread Dominik Vogt
ing "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) Ok. > - > return t; Ok. > mtune= > -Target RejectNegative Joined Enum(processor_type) Var(s390_tune) > Init(PROCESSOR_max) Save. > +Target RejectNegative Joined Enum(processor_type) Var(s390_tune) > Init(PROCESSOR_max) Save > Schedule code for given CPU The full stop should be at the end of the help text, of course. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

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

2015-10-26 Thread Dominik Vogt
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: > >

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

2015-10-26 Thread Dominik Vogt
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: > >

Re: [PATCH] Fix PR67443

2015-10-22 Thread Dominik Vogt
ut I'm pessimistic. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: [PATCH] Add new hooks ASM_OUTPUT_START_FUNCTION_HEADER ...

2015-10-16 Thread Dominik Vogt
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

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

2015-10-16 Thread Dominik Vogt
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: > >

Re: [PATCH] Fix "#pragma GCC pop_options" warning.

2015-10-13 Thread Dominik Vogt
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

[PATCH] Fix "#pragma GCC pop_options" warning.

2015-10-13 Thread Dominik Vogt
ptions): 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 d149dd8b9d6c9f720809de3839f2ad5a6825

Re: [PATCH] Fix "#pragma GCC pop_options" warning.

2015-10-13 Thread Dominik Vogt
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

Re: [PATCH] Add new hooks ASM_OUTPUT_START_FUNCTION_HEADER ...

2015-10-01 Thread Dominik Vogt
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 directives

Re: [PATCH] Add new hooks ASM_OUTPUT_START_FUNCTION_HEADER ...

2015-09-28 Thread Dominik Vogt
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 for;

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

2015-09-25 Thread Dominik Vogt
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: > >

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

2015-09-25 Thread Dominik Vogt
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

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

2015-09-25 Thread Dominik Vogt
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: > >

Re: [PATCH] Add new hooks ASM_OUTPUT_START_FUNCTION_HEADER ...

2015-09-24 Thread Dominik Vogt
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.

Re: patch for PR61578

2015-09-23 Thread Dominik Vogt
with the same value from single 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

Re: [PATCH] Add new hooks ASM_OUTPUT_START_FUNCTION_HEADER ...

2015-09-23 Thread Dominik Vogt
ges 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_function): C

[PATCH] Add new hooks ASM_OUTPUT_START_FUNCTION_HEADER ...

2015-09-21 Thread Dominik Vogt
g 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_function)

[FYI][PATCH] libgo/runtime: S/390: Workaround for broken setcontext in Gcc.

2015-09-01 Thread Dominik Vogt
this patch. ** Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany >From 0ef4dea9441ea53f12040a2e59699b93f1e3913d Mon Sep 17 00:00:00 2001 From: Dominik Vogt <v...@linux.vnet.ibm.com> Date: Mon, 31 Aug 2015 16:21:28 +0100 Subject: [PATCH] libgo/runtime: S/390: Workaround for broken s

[PATCH] Remove reference to undefined documentation node.

2015-08-26 Thread Dominik Vogt
with that change. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * doc/extend.texi: Remove reference to undefined node. From 55b9c29f73d8da1881ce5a3f65d0c7f40623e161 Mon Sep 17 00:00:00 2001 From: Dominik Vogt v...@linux.vnet.ibm.com Date: Wed, 26 Aug 2015 10:59:29 +0100 Subject

Re: [PATCH] Remove reference to undefined documentation node.

2015-08-26 Thread Dominik Vogt
On Wed, Aug 26, 2015 at 11:05:09AM +0100, Dominik Vogt wrote: This patch removes a menu entry that points to an undefined node in the documentation. The faulty entry has been introduced with git commit id 3aabc45f2, subversion id 138bc75d-0d04-0410-96. It looks like the entry is a remnant

[PATCH] s390: Add emit_barrier() after trap.

2015-08-26 Thread Dominik Vogt
^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog * config/s390/s390.c (s390_emit_prologue): Add emit_barrier() after trap to fix ICE. gcc/testsuite/ChangeLog * gcc.target/s390/20150826-1.c: New test. From ec6b88cd51234d138bd559271def086156fcae07 Mon Sep 17 00:00:00 2001 From

[PATCH] Add extra compile options for dg-final only once.

2015-08-20 Thread Dominik Vogt
this be committed? Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/testsuite/ChangeLog * lib/gcc-dg.exp: Add extra options for db-final to the command line only once. From e89aecf367ffd2e89ac6eec7a04edd2eddd2a0da Mon Sep 17 00:00:00 2001 From: Dominik Vogt v...@linux.vnet.ibm.com

Re: [PATCH] S390: Fix vec_load_bndry.

2015-08-20 Thread Dominik Vogt
On Tue, Aug 18, 2015 at 02:49:30PM +0200, Ulrich Weigand wrote: Dominik Vogt wrote: The attached patch fixes the vec_load_bndry builtin on S390. The second argument must be one of 64, 128, 256, ..., 4096, but the table expected that value to fit into 3 bits. Makes sense. However, I'd

[PATCH] S390: Fix vec_load_bndry.

2015-08-18 Thread Dominik Vogt
The attached patch fixes the vec_load_bndry builtin on S390. The second argument must be one of 64, 128, 256, ..., 4096, but the table expected that value to fit into 3 bits. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany From 51d68bc3b6fd293504ff859763bbc4f3aff8cb69 Mon Sep 17 00:00:00

[PATCH] Honour DriverOnly for enum values in error messages.

2015-07-27 Thread Dominik Vogt
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 Germany /gcc/ChangeLog

Re: s390-linux fails to build

2015-07-24 Thread Dominik Vogt
. (Andreas will commit them shortly.) Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

[PATCH] S390: Clean up cross-compile for S390.

2015-07-24 Thread Dominik Vogt
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 Vogt v

Re: [PATCH 0/4] S390 -march=native related fixes

2015-07-24 Thread Dominik Vogt
. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany

Re: s390-linux fails to build

2015-07-23 Thread Dominik Vogt
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

<    1   2   3   4   >