Re: [PATCH] FMA_EXPR can cause -fnon-call-exceptions with floating point args (PR middle-end/79396)

2017-02-27 Thread Richard Biener
On Sat, 25 Feb 2017, Jakub Jelinek wrote: > On Sat, Feb 25, 2017 at 08:42:33AM +0100, Richard Biener wrote: > > On February 24, 2017 9:56:25 PM GMT+01:00, Jakub Jelinek > > wrote: > > >Hi! > > > > > >On the following testcase we replace a PLUS_EXPR (which is considered > > >throwing with -fnon-c

Re: [BUILDROBOT] arm-netbsdelf: Error during -fself-test (was: [PATCH] TS_OPTIMIZATION/TS_TARGET_OPTION need no chain/type)

2017-02-27 Thread Richard Biener
On Mon, 27 Feb 2017, Jan-Benedict Glaw wrote: > Hi Richard, > > On Wed, 2017-01-11 16:28:33 +0100, Richard Biener wrote: > > On Wed, 11 Jan 2017, Richard Biener wrote: > > > LTO bootstrapped on x86_64-unknown-linux-gnu, testing in progress. > > > > > > (most "gross" are still TS_LIST having a t

[wwwdocs] gcc-4.8/changes.html

2017-02-27 Thread Gerald Pfeifer
Somehow this one got lost on my end, until this morning. Embarrassing. :-( Still, since I had prepared it back then, and it was a real change, committed now. Gerald Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/c

Re: enable -Wformat-truncation with -Og (PR 79691)

2017-02-27 Thread Richard Biener
On Fri, Feb 24, 2017 at 6:51 PM, Martin Sebor wrote: > On 02/24/2017 03:10 AM, Richard Biener wrote: >> >> On Fri, Feb 24, 2017 at 1:35 AM, Martin Sebor wrote: >>> >>> Bug 79691 - -Wformat-truncation suppressed by (and only by) -Og >>> points out that the gimple-ssa-sprintf pass doesn't run when

Re: [PATCH 4/5] Document warning option -Wstring-plus-int

2017-02-27 Thread Xi Ruoyao
On 2017-02-25 Sat 18:19 +0100, Gerald Pfeifer wrote: > I.e., after reading the above, I was wondering who'd do "abc" + 1. > > Of course, once I looked at your good additions to the testsuite, > I realized that this is really more about "abc" + '1'.  Technically > the description is correct, since

Re: [gomp4] DEV-PHASE change

2017-02-27 Thread Thomas Schwinge
Hi Tom! On Thu, 23 Feb 2017 17:19:37 +0100, Tom de Vries wrote: > [ was: r241221 [1/2] - in /branches/gomp-4_0-branch: ] > > On 16/10/16 22:13, tschwi...@gcc.gnu.org wrote: > > Author: tschwinge > > Date: Sun Oct 16 20:13:18 2016 > > New Revision: 241221 > > > > URL: https://gcc.gnu.org/vie

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-27 Thread Richard Biener
On Wed, Feb 22, 2017 at 11:11 AM, Martin Jambor wrote: > Hello, > > this is a fix for PR 78140 which is about LTO WPA of Firefox taking > 1GB memory more than gcc 6. > > It works by reusing the ipa_bits and value_range that we previously > had directly in jump functions and which are just too big

Re: [gomp4] DEV-PHASE change

2017-02-27 Thread Tom de Vries
On 27/02/17 10:09, Thomas Schwinge wrote: Hi Tom! On Thu, 23 Feb 2017 17:19:37 +0100, Tom de Vries wrote: [ was: r241221 [1/2] - in /branches/gomp-4_0-branch: ] On 16/10/16 22:13, tschwi...@gcc.gnu.org wrote: Author: tschwinge Date: Sun Oct 16 20:13:18 2016 New Revision: 241221 URL: ht

Re: [PATCH] Some more translation related tweaks

2017-02-27 Thread Volker Reichelt
On 26 Feb, Jakub Jelinek wrote: > On Sun, Feb 26, 2017 at 01:18:57PM +0100, Volker Reichelt wrote: >> 2017-02-26 Volker Reichelt >> >> * init.c: Include intl.h > > Missing . Indeed, I noticed that one after I hit the send button. >> @@ -29,6 +29,7 @@ >> #include "varasm.h" >> #include

Re: [PATCH] Some more translation related tweaks

2017-02-27 Thread Jakub Jelinek
On Mon, Feb 27, 2017 at 11:04:36AM +0100, Volker Reichelt wrote: > > This is not -Wformat-security friendly, perhaps better > > pedwarn (EXPR_LOC_OR_LOC (outer_nelts, input_location), OPT_Wvla, > >typedef_variant_p (orig_type) > >? "non-constant array new lengt

[Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-02-27 Thread Janne Blomqvist
Don't try to use rand_s on CYGWIN CYGWIN seems to include _mingw.h and thus __MINGW64_VERSION_MAJOR is defined even though rand_s is not available. Thus add an extra check for __CYGWIN__. Thanks to Tim Prince and Nightstrike for bringing this issue to my attention. Committed as r245755. 2017-02

Re: [PATCH PR69564]Improve vectorizer's runtime alias check for wrapping type

2017-02-27 Thread Bin.Cheng
On Fri, Feb 24, 2017 at 12:34 PM, Richard Biener wrote: > On Fri, Feb 24, 2017 at 11:53 AM, Bin Cheng wrote: >> Hi, >> As analyzed in PR69564, inefficient code for runtime alias check is >> generated in benchmark >> scimark2. It is suspected vectorized loop doesn't run enough iterations to >>

Re: [PATCH] Some more translation related tweaks

2017-02-27 Thread Joseph Myers
On Mon, 27 Feb 2017, Jakub Jelinek wrote: > On Mon, Feb 27, 2017 at 11:04:36AM +0100, Volker Reichelt wrote: > > > This is not -Wformat-security friendly, perhaps better > > > pedwarn (EXPR_LOC_OR_LOC (outer_nelts, input_location), OPT_Wvla, > > > typedef_variant_p (orig_type) > >

Re: [PATCH] Some more translation related tweaks

2017-02-27 Thread Jakub Jelinek
On Mon, Feb 27, 2017 at 11:04:36AM +0100, Volker Reichelt wrote: > So here's the second attempt: > > 2017-02-27 Volker Reichelt > > * init.c: Include intl.h. > (build_new_1): Move message strings into pedwarn to make them > -Wformat-security friendly. Mark string for translat

Re: [PATCH] Some more translation related tweaks

2017-02-27 Thread Volker Reichelt
On 27 Feb, Jakub Jelinek wrote: > On Mon, Feb 27, 2017 at 11:04:36AM +0100, Volker Reichelt wrote: >> So here's the second attempt: >> >> 2017-02-27 Volker Reichelt >> >> * init.c: Include intl.h. >> (build_new_1): Move message strings into pedwarn to make them >> -Wformat-secur

C++ PATCH to fix a comment

2017-02-27 Thread Marek Polacek
You might think it doesn't matter, but it causes vim's "jump to matching brace" feature to malfunction, leaving me a bit miffed. Applying to trunk. 2017-02-27 Marek Polacek * decl.c (expand_static_init): Add missing } in a comment. diff --git gcc/cp/decl.c gcc/cp/decl.c index 3dd1f5c

RE: [PATCH] [ARC] Clean up arc header file.

2017-02-27 Thread Claudiu Zissulescu
> Oh, and you'll need to update the copyright headers to 2017 when you > do eventually apply this, but everything else looks fine. > Forgot to commit this one, done in this commit r245759. Sorry for this, Claudiu

[PATCH] Fix PR79723

2017-02-27 Thread Richard Biener
I am testing the following patch for another vectorizer issue with address-space preserving. Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2017-02-27 Richard Biener PR tree-optimization/79723 * tree-vect-stmts.c (get_vectype_for_scalar_type_and_size): Pre

[gomp4] add support for gang local storage allocation in shared memory

2017-02-27 Thread Cesar Philippidis
This patch, which is largely implemented by Chung-Lin, is a first step towards teaching the c and c++ FEs how to allocate shared memory for gang local variables. E.g. #pragma acc parallel { int some_array[N], some_var; Both some_array and some_var will be stored in shared memory with this

New template for 'gcc' made available

2017-02-27 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: http://translationproject.org/POT-files/gcc-

Re: [PATCH] avoid using upper bound of width and precision in -Wformat-overlow (PR 79692)

2017-02-27 Thread Jeff Law
On 02/25/2017 03:46 PM, Martin Sebor wrote: In an arithmetic directive with the width or precision specified by an argument to the asterisk (e.g., "%*x") and where the argument range is unknown, for the purposes of the return value optimization the pass must assume it's potentially as large as IN

Re: [wwwdocs] Document in changes.html -fcode-hoisting, -fipa-bit-cp, -fipa-vrp, -fsplit-loops, GCJ removal, x86 ISA additions, -fshrink-wrap-separate etc.

2017-02-27 Thread Martin Sebor
On 02/26/2017 07:21 AM, Gerald Pfeifer wrote: On Sun, 19 Feb 2017, Gerald Pfeifer wrote: That was quite a bit; thanks for doing that, Jakub! In the patch below I try to streamline language a bit, document options being implied by -Os (in addition to -O2 or higher), fix grammar in a few places,

[RFC PATCH 0/3] Call summary class

2017-02-27 Thread Martin Jambor
Hello, the patch sequence in this thread adds a call_summary class, which is analogous to function_summary we already have but which gathers information about call graph edges, rather than nodes. The first patch implements the class itself, the second modifies ipa-prop.[ch] and ipa-cp.c to use it

[RFC PATCH 1/3] call_summary to keep info about cgraph_edges

2017-02-27 Thread Martin Jambor
Hello, this patch is an actual implementation of the call_summary class (I hope the name is a good analogy to the function_summary we have, I am opened to other suggestions). I have kept the implementation close to the existing one of function-summary, there are I think only two notable differenc

[RFC PATCH 3/3] Remove ipa_update_after_lto_read

2017-02-27 Thread Martin Jambor
Hello, when working on call summaries, I have found a weird function ipa_update_after_lto_read which currently only makes sure that IPA-CP function and edge summaries exist, at times when they already have to exist. So I did some digging in history and found out that originally the function was m

[RFC PATCH 2/3] Use call_summary in ipa-prop and ipa-cp

2017-02-27 Thread Martin Jambor
Hello, this is patch is afairly straightforward conversion from use of a vector indexed by edge->uid to use of the new call_summary from the previous patch. The patch is generally a cleanup, hashing is a nicer method of keeping call-site related information than a gigantic vector that we never sh

Re: [wwwdocs] Document in changes.html -fcode-hoisting, -fipa-bit-cp, -fipa-vrp, -fsplit-loops, GCJ removal, x86 ISA additions, -fshrink-wrap-separate etc.

2017-02-27 Thread Martin Jambor
Hello, On Sun, Feb 26, 2017 at 03:21:20PM +0100, Gerald Pfeifer wrote: > On Sun, 19 Feb 2017, Gerald Pfeifer wrote: > > That was quite a bit; thanks for doing that, Jakub! > > > > In the patch below I try to streamline language a bit, document options > > being implied by -Os (in addition to -O2

Re: [PATCH] Remove x86 pcommit instruction

2017-02-27 Thread Andrew Senkevich
2017-02-25 18:10 GMT+01:00 Gerald Pfeifer : > On Wed, 7 Dec 2016, Andrew Senkevich wrote: But how to update changes for upcoming GCC 6.3 (and future GCC 5.5)? >>> There are corresponding documents at [1] and [2], please add a "GCC >>> 6.3" or "GCC 5.5" entry with "Target Specific Changes" at

Re: [PATCH] Some more translation related tweaks

2017-02-27 Thread Jakub Jelinek
On Mon, Feb 27, 2017 at 12:47:09PM +, Joseph Myers wrote: > On Mon, 27 Feb 2017, Jakub Jelinek wrote: > > > On Mon, Feb 27, 2017 at 11:04:36AM +0100, Volker Reichelt wrote: > > > > This is not -Wformat-security friendly, perhaps better > > > > pedwarn (EXPR_LOC_OR_LOC (outer_nelts, i

[Patch, fortran] PR79676 - [submodules] Compilation/linking error when module procedures PRIVATE

2017-02-27 Thread Paul Richard Thomas
Dear All, This bug resulted from a cock-up on my part. The mechanism for suppressing .smod files depended on detecting the presence of a module procedure by resetting a flag if the module_procedure attribute was written. Of course, this didn't happen if the module procedure is private, which rathe

Disable some concept checks in C++11

2017-02-27 Thread François Dumont
Hi I had some problems when testing pretty printers while having activated concept checks. I noticed that std::deque had already the _SGIAssignableConcept check disable when using C++11 so I propose to generalize to all usages of this concept check. Additionally it avoids the defini

Re: [PATCH] Some more translation related tweaks

2017-02-27 Thread Joseph Myers
On Mon, 27 Feb 2017, Jakub Jelinek wrote: > > Yes, it's generally the case that G_() is used whenever there's a > > conditional expression for the msgid argument to a diagnostic function. > > So, is this ok for trunk? Shall I regenerate gcc.pot or will you? OK. I'll regenerate gcc.pot in a fe

Re: Documentation patch related to PR42376

2017-02-27 Thread Gerald Pfeifer
On Mon, 11 Jan 2010, Martin Reinecke wrote: > thanks for applying the patch! > Unfortunately I just noticed some mistakes in my wording :( > Currently the sentence says > "code [...] may run significantly slower [...] than they did [...]" > I'm not a native speaker, but I guess it would be more cor

Re: [wwwdocs] Update changes.html for LTO and IPA

2017-02-27 Thread Gerald Pfeifer
On Wed, 20 Jan 2016, Jan Hubicka wrote: > this is updated patch. I tried to explain better the situation WRT > incremental linking. Thank you, Jan. I had a couple of editorial changes on top of this, which I finally managed to commit. (See the patch below.) And one question: "declaration linki

Re: [wwwdocs] Document in changes.html -fcode-hoisting, -fipa-bit-cp, -fipa-vrp, -fsplit-loops, GCJ removal, x86 ISA additions, -fshrink-wrap-separate etc.

2017-02-27 Thread Gerald Pfeifer
Hi Martin, at first I was surprised to see two suggestions from -- until I realized those came from the two (Martins) of you. ;-) On Mon, 27 Feb 2017, Martin Jambor wrote: > on top of your patch, I would like to propose the following. What do > you think? > > Index: changes.html >

Re: [PATCH] Remove x86 pcommit instruction

2017-02-27 Thread Gerald Pfeifer
Hi Andrew, On Mon, 27 Feb 2017, Andrew Senkevich wrote: >> +Support for the > + >> href="https://software.intel.com/en-us/blogs/2016/09/12/deprecate-pcommit-instruction";>deprecated >> +pcommit instruction has been removed. > Could you please add the same statement for gcc-7 changes?

Re: [PATCH] rs6000: float128 on BE and 32-bit

2017-02-27 Thread Michael Meissner
On Mon, Feb 13, 2017 at 10:20:48PM +, Segher Boessenkool wrote: > This fixes float128 on BE and on 32-bit. > > The configure tests need to use -mabi=altivec for 32-bit, since it is > not the default there. That also enables the "vector" keyword, used by > the tests. To do this it temporarily

Re: Documentation patch related to PR42376

2017-02-27 Thread Sandra Loosemore
On 02/27/2017 03:26 PM, Gerald Pfeifer wrote: On Mon, 11 Jan 2010, Martin Reinecke wrote: thanks for applying the patch! Unfortunately I just noticed some mistakes in my wording :( Currently the sentence says "code [...] may run significantly slower [...] than they did [...]" I'm not a native sp

[wwwdocs] adding ira-select branch description

2017-02-27 Thread Vladimir Makarov
Gerald reminded me that it would be nice to add ira-select branch description to svn.html. Here is the patch I've committed. Index: svn.html === RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v retrieving revision 1.215 diff -u -r1.215

[wwwdocs] Handle /onlinedocs/ld

2017-02-27 Thread Gerald Pfeifer
>From the GCC documentation under /onlinedocs there is a link to /onlinedocs/ld. That does not exist on gcc.gnu.org, so provide a redirect to where it actually is available. Committed. Gerald Index: .htaccess === RCS file: /cvs/gc

gcc-7/changes.html adjustments

2017-02-27 Thread Richard Biener
Committed. Richard. 2017-02-28 Richard Biener * gcc-7/changes.html: Fix double value, adjust -Wstringop-overflow default. Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-7/changes.html,v retrieving