Re: [committed] Don't suppress bogus usage of macros from system headers in -Wformat (PR c/78304)

2017-01-16 Thread Christophe Lyon
Hi David, On 13 January 2017 at 21:04, David Malcolm wrote: > c-lex.c: lex_string uses cpp_get_token rather than > cpp_get_token_with_location, and hence the C family of frontends > record the physical locations of tokens in string concatenations, rather > than the virtual

Re: [PATCH] Fix gcc.dg/tree-ssa/scev-[345].c testcases

2017-01-16 Thread Rainer Orth
Hi Richard, > The following is an attempt to change those testcases to be less dependent > on previous passes. The original motivation of the testcases seems to be > testing SCEV capabilities and in turn IVOPTs decisions, thus the testcases > are changed to check the IVO dump, use the GIMPLE FE

[Bug tree-optimization/79054] missing range information with INT_MAX

2017-01-16 Thread clyon at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79054 Christophe Lyon changed: What|Removed |Added Target|poerpc64*-*-* |powerpc64*-*-* CC|

Re: [PATCH] Fix gcc.dg/tree-ssa/scev-[345].c testcases

2017-01-16 Thread Richard Biener
On Mon, 16 Jan 2017, Christophe Lyon wrote: > On 13 January 2017 at 12:16, Bin.Cheng wrote: > > On Fri, Jan 13, 2017 at 9:46 AM, Richard Biener wrote: > >> > >> The following is an attempt to change those testcases to be less dependent > >> on previous

[PATCH] Fix PR71433

2017-01-16 Thread Richard Biener
The following makes VRP deal better with the situation where the same assertion is to be inserted on all predecessors of a BB. That avoids the spurious array-bound warning (and enables some optimization) when jump-threading f**ed up the CFG in such redundant way. (the fully "correct" way to deal

Re: [PATCH] BRIG frontend: request for a global review

2017-01-16 Thread Pekka Jääskeläinen
OK, I'll see into adapting the Jakub's idea and also check if some of the simplest builtins are better expanded directly to tree nodes instead. I'm not sure if lto support is needed though as the assumption now is to have fully linked input to this FE (all necessary BRIG modules fed in at build

Re: [PATCH] Fix gcc.dg/tree-ssa/scev-[345].c testcases

2017-01-16 Thread Christophe Lyon
On 13 January 2017 at 12:16, Bin.Cheng wrote: > On Fri, Jan 13, 2017 at 9:46 AM, Richard Biener wrote: >> >> The following is an attempt to change those testcases to be less dependent >> on previous passes. The original motivation of the testcases seems

[Bug tree-optimization/77283] [7 Regression] Revision 238005 disables loop unrolling

2017-01-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77283 --- Comment #19 from Richard Biener --- Author: rguenth Date: Mon Jan 16 09:33:12 2017 New Revision: 244487 URL: https://gcc.gnu.org/viewcvs?rev=244487=gcc=rev Log: 2017-01-13 Richard Biener PR

[Bug middle-end/79089] [7 Regression] error: incorrect sharing of tree nodes

2017-01-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79089 Jakub Jelinek changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

[PATCH] Make multiple_target.c aware of LTO (PR lto/66295)

2017-01-16 Thread Martin Liška
Hello. Not being expert in multi_target area, however it consists of 2 passes. The first one (ipa_target_clone) is responsible for creation of multiple targets for functions decorated with __attribute__((target_clones("xxx"))). I guess the pass should be called just in LGEN phase and

[Bug target/79058] [7 Regression] ARM: internal compiler error: in extract_constrain_insn, at recog.c:2213

2017-01-16 Thread vogt at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79058 --- Comment #22 from Dominik Vogt --- That looks like a similar problem. I'm lacking some knowledge about how register pairs are allocated for paradoxical subregs on bigendian systems though. Deducing from the code quoted above and from what

[Bug demangler/79099] Memory leak in cplus-dem.c:2232

2017-01-16 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79099 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/71737] [5/6 Regression] ICE following 2x pack expansion in non-pack with template alias

2017-01-16 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71737 Paolo Carlini changed: What|Removed |Added Summary|[5/6/7 Regression] ICE |[5/6 Regression] ICE

[Bug c++/71737] [5/6/7 Regression] ICE following 2x pack expansion in non-pack with template alias

2017-01-16 Thread paolo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71737 --- Comment #6 from paolo at gcc dot gnu.org --- Author: paolo Date: Mon Jan 16 09:09:30 2017 New Revision: 244486 URL: https://gcc.gnu.org/viewcvs?rev=244486=gcc=rev Log: /c-family 2017-01-16 Paolo Carlini

Re: [PATCH] BRIG frontend: request for a global review

2017-01-16 Thread Jakub Jelinek
On Mon, Jan 16, 2017 at 09:46:43AM +0100, Richard Biener wrote: > There are 187 of them (well, simple grep of DEF_HSAIL, so probably a bit > less). > They aren't really documented but I guess that __hsail_bitmask_u64 for example > is really equivalent to sth like -1U >> n << m? So I'm not sure

[Bug bootstrap/79098] [7 Regression] Failure to bootstrap gcc for powerpc-e500v2-linux-gnuspe target

2017-01-16 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79098 Markus Trippelsdorf changed: What|Removed |Added Priority|P3 |P1

[Bug bootstrap/79098] [7 Regression] Failure to bootstrap gcc for powerpc-e500v2-linux-gnuspe target

2017-01-16 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79098 Markus Trippelsdorf changed: What|Removed |Added Target|powerpc-e500v2-linux-gnuspe |powerpc-e500v2-linux-gnuspe

Re: [PATCH][PR tree-optimization/79090] Fix two minor DSE bugs

2017-01-16 Thread Richard Biener
On Sun, Jan 15, 2017 at 10:34 AM, Jeff Law wrote: > > At one time I know I had the max_size == size test in valid_ao_ref_for_dse. > But it got lost at some point. This is what caused the Ada failure. > > Technically it'd be OK for the potentially dead store to have a variable >

[Bug lto/79061] [7 Regression][LTO][ASAN] LTO plus ASAN fails with "AddressSanitizer: initialization-order-fiasco"

2017-01-16 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79061 --- Comment #9 from Jakub Jelinek --- I think it is fine if it has DECL_NAME NULL, but it would be helpful if DECL_SOURCE_LOCATION of the TRANSLATION_UNIT_DECL was set to some location in the main input file (e.g. first column on first line of

Re: [PATCH] BRIG frontend: request for a global review

2017-01-16 Thread Richard Biener
On Fri, Jan 13, 2017 at 4:54 PM, Pekka Jääskeläinen wrote: > On Fri, Jan 13, 2017 at 2:34 PM, Richard Biener > wrote: >> On Thu, Jan 12, 2017 at 3:55 PM, Pekka Jääskeläinen >> wrote: >>> Hi, >>> >>> A gentle ping... >> >>

[Bug c++/79091] [7 regression] ICE in write_unnamed_type

2017-01-16 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79091 Martin Liška changed: What|Removed |Added CC||jason at gcc dot gnu.org,

[Bug lto/79061] [7 Regression][LTO][ASAN] LTO plus ASAN fails with "AddressSanitizer: initialization-order-fiasco"

2017-01-16 Thread m.ostapenko at samsung dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79061 --- Comment #8 from Maxim Ostapenko --- (In reply to Jakub Jelinek from comment #7) > Comment on attachment 40514 [details] > Untested fix 1. > > But DECL_SOURCE_FILE is not the main input file of the TU that contains it, > if e.g. a variable

Re: [PATCH] system.h: Poison strndup (PR bootstrap/78616)

2017-01-16 Thread Richard Biener
On Fri, Jan 13, 2017 at 10:17 PM, David Malcolm wrote: > This patch poisons strndup (in system.h), as requested in the > discussion of PR bootstrap/78616. > > Successfully bootstrapped on x86_64-pc-linux-gnu. > > OK for trunk? Ok. Richard. > gcc/ChangeLog: > PR

[ping] 3 aarch64/arm/rs6000 patches

2017-01-16 Thread Eric Botcazou
aarch64 (Enable descriptors for nested functions in Ada): https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01253.html arm (Enable descriptors for nested functions in Ada): https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01254.html rs6000 (Fix reload failures in 64-bit mode):

[Bug tree-optimization/32306] [5/6/7 Regression] redundant && || not eliminated

2017-01-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=32306 Richard Biener changed: What|Removed |Added Status|WAITING |NEW --- Comment #32 from Richard

[Bug c++/79085] ICE with placement new to unaligned location

2017-01-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79085 Richard Biener changed: What|Removed |Added Target||arm-none-eabi

[Bug tree-optimization/32199] jc1: out of memory allocating 4072 bytes after a total of 805021000 bytes

2017-01-16 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=32199 Richard Biener changed: What|Removed |Added Resolution|WORKSFORME |FIXED --- Comment #20 from Richard

Re: [patch,avr]: Increase branch costs after reload.

2017-01-16 Thread Denis Chertykov
2017-01-13 17:28 GMT+04:00 Georg-Johann Lay : > This adds a penalty of 4 to the post-reload branch costs. > > Purpose is reduce the number of out-of-line blocks like in > > unsigned long variant5 (unsigned in) > { > unsigned long out = 0; > if (in & (1 << 0)) out |= 0xful <<

<    1   2   3