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
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
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
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
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
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
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
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
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.
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
?
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
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
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
the information I've added to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60401
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
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
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:
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
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 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
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
The following two patches fix some test cases for S/390.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
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
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
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 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
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
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
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
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_
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
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
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
&
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
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
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.
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
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
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
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
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
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
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
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 @@
> >>>
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. */
&
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
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
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
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
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 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
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
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
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 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
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 "
(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:
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 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
_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
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 (
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
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:
>
>
ut I'm pessimistic.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
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:
>
>
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
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
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 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
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;
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:
>
>
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 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.
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
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
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)
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
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
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
^_^ ^_^
--
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
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
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
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
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
. (Andreas
will commit them shortly.)
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 Vogt v
.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
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
201 - 300 of 372 matches
Mail list logo