[PATCH, libphobos] Committed fix configure test for backtrace-supported.h

2019-04-15 Thread Iain Buclaw
Hi, When porting/testing the D front-end to FreeBSD, I noticed that backtrace supported returned false during the configuration of libphobos. The use of += assignment in the configure test was the reason why, and now that's been corrected. Bootstrapped and regression tested on x86_64-linux-gnu a

Re: [PATCH wwwdocs] Mention GNU Tools Cauldron in the News section

2019-04-15 Thread Eric Gallager
On 4/15/19, Simon Marchi wrote: > Hi, > > Here is a patch that adds a mention of the 2019 Cauldron, similar to the > entries for the previous editions. > > Thanks, > > Simon > > > Index: index.html > === > RCS file: /cvs/gcc/wwwdocs/h

New French PO file for 'gcc' (version 9.1-b20190414)

2019-04-15 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the French team of translators. The file is available at: https://translationproject.org/latest/gcc/fr.po (This file, 'gcc-9.1-b20190414.fr.po',

[committed] Fix various microblaze-linux failures

2019-04-15 Thread Jeff Law
microblaze testing in my tester has occasionally been failing Warray-bounds-40 and Wstringop-overflow-9. I finally took a little peek because these occasional failures show up as a regression against the prior run. It looks like the microblaze backend is trying to inline a move of SIZE_MAX bytes.

[PATCH rs6000] Fix PR target/84369: gcc.dg/sms-10.c fails on Power9

2019-04-15 Thread Pat Haugen
As pointed out in the PR, the test is failing because a store->load dependency is reporting zero cost. Fixed by leaving existing costs as is (i.e. cost for update forms), and just adding a simple bypass for store->load dependencies. Bootstrap/regtest on powerpc64le (Power9) with no new regressio

[PATCH, libphobos] Committed merge with upstream druntime

2019-04-15 Thread Iain Buclaw
Hi, This patch merges the libdruntime library with upstream druntime 70b9fea6. Backports fixes in the extern(C) bindings for the Solaris/SPARC port. Bootstrapped and regression tested on x86_64-linux-gnu and i386-pc-solaris2.11. Committed to trunk as r270372. -- Iain --- diff --git a/libphobo

Re: [C++ Patch/RFC] PR 89900 ("[9 Regression] ICE: Segmentation fault (in check_instantiated_arg)")

2019-04-15 Thread Paolo Carlini
Hi, On 12/04/19 20:29, Jason Merrill wrote: On 4/11/19 11:20 AM, Paolo Carlini wrote: Hi, over the last few days I spent some time on this regression, which at first seemed just a minor error-recovery issue, but then I noticed that very slightly tweeking the original testcase uncovered a pre

Re: [patch] Fix PR 84487, large rodata increase in tonto and other programs

2019-04-15 Thread Segher Boessenkool
On Mon, Apr 15, 2019 at 01:54:11PM +0200, Florian Weimer wrote: > * Richard Biener: > > > Of course adding sth like a .robss section would be nice. > > I think this is strictly a link editor issue because a read-only PT_LOAD > directive with a memory size larger than the file size already produce

Re: [PATCH wwwdocs] Mention GNU Tools Cauldron in the News section

2019-04-15 Thread Simon Marchi
On 2019-04-15 12:42 p.m., Simon Marchi wrote: > Hi, > > Here is a patch that adds a mention of the 2019 Cauldron, similar to the > entries > for the previous editions. > > Thanks, > > Simon > > > Index: index.html > === > RCS fil

Re: [PATCH] (RFA tree-tailcall) PR c++/82081 - tail call optimization breaks noexcept

2019-04-15 Thread Andrew Pinski
On Sun, Apr 14, 2019 at 11:50 PM Richard Biener wrote: > > On Sat, Apr 13, 2019 at 12:34 AM Jeff Law wrote: > > > > On 4/12/19 3:24 PM, Jason Merrill wrote: > > > If a noexcept function calls a function that might throw, doing the tail > > > call optimization means that an exception thrown in the

[PATCH wwwdocs] Mention GNU Tools Cauldron in the News section

2019-04-15 Thread Simon Marchi
Hi, Here is a patch that adds a mention of the 2019 Cauldron, similar to the entries for the previous editions. Thanks, Simon Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.1125 diff -u

Re: [PATCH v2] Fix __patchable_function_entries section flags

2019-04-15 Thread Joao Moreira
On 4/12/19 1:19 PM, Jeff Law wrote: On 4/11/19 11:18 AM, Joao Moreira wrote: When -fpatchable-relocation-entry is used, gcc places nops on the prologue of each compiled function and creates a section named __patchable_function_entries which holds relocation entries for the positions in which

[aarch64][RFA/RFC][rtl-optimization/87763] Add new movk pattern for aarch64

2019-04-15 Thread Jeff Law
Here's my attempt to fix the movk regression on bz 87763. I still wonder if addressing some of these issues in combine is a better long term solution, but in the immediate term I think backend patterns are going to have to be the way to go. This introduces a new insn_and_split that matches a mov

Re: [PATCH] Add support for missing AVX512* ISAs (PR target/89929).

2019-04-15 Thread H.J. Lu
On Mon, Apr 15, 2019 at 12:26 AM Martin Liška wrote: > > On 4/12/19 4:12 PM, H.J. Lu wrote: > > On Fri, Apr 12, 2019 at 4:41 AM Martin Liška wrote: > >> > >> On 4/11/19 6:30 PM, H.J. Lu wrote: > >>> On Thu, Apr 11, 2019 at 1:38 AM Martin Liška wrote: > > Hi. > > The patch is

Re: [PATCH] fix ICEs in c-attribs.c (PR 88383, 89288, 89798, 89797)​

2019-04-15 Thread Jeff Law
On 4/15/19 7:12 AM, Christophe Lyon wrote: > On Sat, 13 Apr 2019 at 00:38, Martin Sebor wrote: >> >> On 4/12/19 3:42 PM, Jakub Jelinek wrote: >>> On Fri, Apr 12, 2019 at 10:45:25AM -0600, Jeff Law wrote: > gcc/ChangeLog: > > PR c/89797 > * targhooks.c (default_vector_alignm

[PATCH] Tweak LIM MEM improvements to fix PR56049

2019-04-15 Thread Richard Biener
It turns out solving this long-standing optimization regression is now easy by exploiting implmenetation details in how we canonicalize refs in LIM. This allows us to properly identifying MEM[(integer(kind=4)[64] *)&a][0] and MEM[(c_char * {ref-all})&a] the same, applying store-motion to an init

Re: [PATCH] fix ICEs in c-attribs.c (PR 88383, 89288, 89798, 89797)​

2019-04-15 Thread Christophe Lyon
On Sat, 13 Apr 2019 at 00:38, Martin Sebor wrote: > > On 4/12/19 3:42 PM, Jakub Jelinek wrote: > > On Fri, Apr 12, 2019 at 10:45:25AM -0600, Jeff Law wrote: > >>> gcc/ChangeLog: > >>> > >>> PR c/89797 > >>> * targhooks.c (default_vector_alignment): Avoid assuming > >>> argument fits in

Re: [PATCH] Fix up RTL DCE find_call_stack_args (PR rtl-optimization/89965)

2019-04-15 Thread Michael Matz
Hi, On Fri, 12 Apr 2019, Jeff Law wrote: > > I don't think this follows. Imagine a pure foo tailcalling a pure bar. > > To make the tailcall, foo may need to change some of its argument slots > > to pass new arguments to bar. > I'd claim that a pure/const call can't tail call another function as

Re: [PATCH] Add support for missing AVX512* ISAs (PR target/89929).

2019-04-15 Thread Martin Liška
On 4/12/19 4:12 PM, H.J. Lu wrote: > On Fri, Apr 12, 2019 at 4:41 AM Martin Liška wrote: >> >> On 4/11/19 6:30 PM, H.J. Lu wrote: >>> On Thu, Apr 11, 2019 at 1:38 AM Martin Liška wrote: Hi. The patch is adding missing AVX512 ISAs for target and target_clone attributes. >>

Re: [Patch, fortran] PRs 89843 and 90022 - C Fortran Interop fixes.

2019-04-15 Thread Paul Richard Thomas
Dear Dominique, Gilles and Reinhold, Thank you for your rapid feedback. We might even get a reasonably functional ISO Fortran binding in place for 9-branch release :-) On your remaining nits: (i) ISO_Fortran_binding_4.f90 -m32 -O1/Os looks awful. I will take a look, though. (ii) pr89844 being f

Re: Fix false -Wodr warnings

2019-04-15 Thread Richard Biener
On Sun, Apr 14, 2019 at 10:59 PM Jan Hubicka wrote: > > Hi, > this patch fixes false warning that is output when different -std > settings are used. In this case C++ FE produces same declaration in > different representations which differ by 0 sized fileds only. > The patch makes them to be ignore

Re: [patch] Fix PR 84487, large rodata increase in tonto and other programs

2019-04-15 Thread Florian Weimer
* Richard Biener: > Of course adding sth like a .robss section would be nice. I think this is strictly a link editor issue because a read-only PT_LOAD directive with a memory size larger than the file size already produces read-only zero pages, without requiring a file allocation. Thanks, Floria

Re: [patch] Fix PR 84487, large rodata increase in tonto and other programs

2019-04-15 Thread Jan Hubicka
> > This won't work with LTO. Note we have the issue in the middle-end as well > since we promote variables we see are not written to to TREE_READONLY. > This can be seen with (the somewhat artificial...): > > int a[1024*1024] = { 0 }; > > int __attribute__((noinline)) foo() { return *(volatile

Re: [PATCH] combine: Count auto_inc properly (PR89794)

2019-04-15 Thread Segher Boessenkool
On Sun, Apr 14, 2019 at 09:51:39AM +, Segher Boessenkool wrote: > The code that checks if an auto-increment from i0 or i1 is not lost is > a bit shaky. The code to check the same for i2 is non-existent, and > cannot be implemented in a similar way at all. So, this patch counts > all auto-incr

Re: [PR86438] avoid too-long shift in test

2019-04-15 Thread Andrew Stubbs
On 12/04/2019 02:42, Alexandre Oliva wrote: The test fell back to long long and long when __int128 is not available, but it assumed sizeof(long) < sizeof(long long) because of a shift count that would be out of range for a long long if their widths are the same. Fixed by splitting it up into two

Re: [PATCH] Filter out LTO in config/bootstrap-lto-lean.mk.

2019-04-15 Thread Martin Liška
On 4/15/19 12:23 PM, Richard Biener wrote: > On Mon, Apr 15, 2019 at 9:46 AM Martin Liška wrote: >> >> Hi. >> >> The patch is fixing bootstrap-lto-lean.mk where with PGO LTO was >> wrongly used in STAGEtrain. >> >> Tested on openSUSE gcc9 package, I'm attaching build log. >> >> Ready to be install

Re: [PATCH] Filter out LTO in config/bootstrap-lto-lean.mk.

2019-04-15 Thread Richard Biener
On Mon, Apr 15, 2019 at 9:46 AM Martin Liška wrote: > > Hi. > > The patch is fixing bootstrap-lto-lean.mk where with PGO LTO was > wrongly used in STAGEtrain. > > Tested on openSUSE gcc9 package, I'm attaching build log. > > Ready to be installed? I wonder why 'override' is necessary given before

Re: [PATCH] Fix PR88936

2019-04-15 Thread Richard Biener
On Fri, 12 Apr 2019, Richard Biener wrote: > On Fri, 12 Apr 2019, Richard Biener wrote: > > > On Fri, 12 Apr 2019, Michael Matz wrote: > > > > > Hi, > > > > > > On Fri, 12 Apr 2019, Richard Biener wrote: > > > > > > > > You miss PARM_DECLs and RESULT_DECLs, i.e. it's probably better to > > >

[PATCH] Fix PR90071

2019-04-15 Thread Richard Biener
The following fixes reassoc leaking abnormals into rewritten conditon chains. Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2019-04-15 Richard Biener PR tree-optimization/90071 * tree-ssa-reassoc.c (init_range_entry): Do not pick up abnormal oper

[PATCH] Fix PR90074

2019-04-15 Thread Richard Biener
I am testing the following patch to fix wrong-debug creatd by loop-distribution simply dropping debug stmts on the floor making earlier ones with bogus value live. Bootstrap & regtest running on x86_64-unknown-linux-gnu. Richard. 2019-04-15 Richard Biener PR debug/90074 * t

AW: [Patch, fortran] PRs 89843 and 90022 - C Fortran Interop fixes.

2019-04-15 Thread Bader, Reinhold
Dear Paul, mostly looks good. Apart from a regression with optional arguments reported as https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90093 all other test cases I have now execute correctly. Cheers Reinhold > -Ursprüngliche Nachricht- > Von: Paul Richard Thomas > Gesendet: Sonntag,

New Spanish PO file for 'gcc' (version 9.1-b20190414)

2019-04-15 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Spanish team of translators. The file is available at: https://translationproject.org/latest/gcc/es.po (This file, 'gcc-9.1-b20190414.es.po',

New template for 'gcc' made available

2019-04-15 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. (If you have any questions, send them to .) A new POT file for textual domain 'gcc' has been made available to the language teams for translation. It is archived as: https://translationproject.org/POT-files/gcc

Re: [Patch, fortran] PRs 89843 and 90022 - C Fortran Interop fixes.

2019-04-15 Thread Dominique d'Humières
Hi Paul, I have found another glitch with -m32 and -O1 or -Os, but not with other values: % gfc /opt/gcc/_clean/gcc/testsuite/gfortran.dg/ISO_Fortran_binding_4.f90 -m32 -O % ./a.out FAIL Note: The following floating-point exceptions are signalling: IEEE_DENORMAL STOP 1 This looks tricky: if I

[PATCH committed] [Bug tree-optimization/90020] [7/8 regression] -O2 -Os x86-64 wrong code generated for GNU Emacs

2019-04-15 Thread Dominique d'Humières
Author: dominiq Date: Mon Apr 15 07:56:43 2019 New Revision: 270360 URL: https://gcc.gnu.org/viewcvs?rev=270360&root=gcc&view=rev Log: 2019-04-15 Dominique d'Humieres PR tree-optimization/90020 * gcc.dg/torture/pr90020.c: Add linker options for darwin. --- trunk/gcc/testsuite/gc

Re: [PATCH] Reset proper type on vector types (PR middle-end/88587).

2019-04-15 Thread Martin Liška
On 4/15/19 9:27 AM, Richard Biener wrote: > On Mon, Apr 15, 2019 at 8:48 AM Martin Liška wrote: >> >> Hi. >> >> Apparently, there's one another PR: >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90083 >> >> May I backport the patch to GCC-8 branch? > > Hmm, it isn't a regression, right? But it

[PATCH] Filter out LTO in config/bootstrap-lto-lean.mk.

2019-04-15 Thread Martin Liška
Hi. The patch is fixing bootstrap-lto-lean.mk where with PGO LTO was wrongly used in STAGEtrain. Tested on openSUSE gcc9 package, I'm attaching build log: https://drive.google.com/file/d/17sxGf_x_VaUekPk2SHI9joIXg1BR5-dY/view?usp=sharing Ready to be installed? Thanks, Martin config/ChangeLog:

Re: [PATCH] Reset proper type on vector types (PR middle-end/88587).

2019-04-15 Thread Richard Biener
On Mon, Apr 15, 2019 at 8:48 AM Martin Liška wrote: > > Hi. > > Apparently, there's one another PR: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90083 > > May I backport the patch to GCC-8 branch? Hmm, it isn't a regression, right? But it only affects multi-versioning, so yes, go ahead. Might

Re: [PATCH] Add support for missing AVX512* ISAs (PR target/89929).

2019-04-15 Thread Martin Liška
On 4/12/19 4:12 PM, H.J. Lu wrote: > On Fri, Apr 12, 2019 at 4:41 AM Martin Liška wrote: >> >> On 4/11/19 6:30 PM, H.J. Lu wrote: >>> On Thu, Apr 11, 2019 at 1:38 AM Martin Liška wrote: Hi. The patch is adding missing AVX512 ISAs for target and target_clone attributes. >>

Re: [patch] Fix PR 84487, large rodata increase in tonto and other programs

2019-04-15 Thread Richard Biener
On Sat, Apr 13, 2019 at 8:48 PM Thomas Koenig wrote: > > Hello world, > > the attached patch fixes a 8/9 regression where _def_init, an internal > Fortran variable containing only zeros, was placed into the .rodata > section. This led to a large increase in executable size. > > There should be no