[Bug preprocessor/80716] Death by Documentation (SEGFAULT)

2017-05-11 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80716 --- Comment #3 from Jakub Jelinek --- Of course, if you want to have \s at the end of line in comments, you just should use /* ... */ style comments that can span multiple lines.

[Bug preprocessor/80716] Death by Documentation (SEGFAULT)

2017-05-11 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80716 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #2

[Bug fortran/79440] internal compiler error: in fold_convert_loc, at fold-const.c:2373

2017-05-11 Thread berke.durak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79440 Berke Durak changed: What|Removed |Added CC||berke.durak at gmail dot com --- Comment

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Bernhard Reutner-Fischer
On 12 May 2017 02:29:58 CEST, Nathan Sidwell wrote: >On 05/11/2017 06:52 PM, Bernhard Reutner-Fischer wrote: >> On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: > >> Anyway, please also adjust the dump cleanup to handle class or I can >take care of it sometimes

[gomp4] correct an implicit data clause bug

2017-05-11 Thread Cesar Philippidis
I introduced a subtle bug in my fix for pr70828. That PR involved implicit data mappings of subarrays which the user had explicitly mapped inside an enclosing acc data region. E.g. #pragma acc data copy(a[5:10]) { #pragma acc parallel loop for (...) a[i] = ... My fix for the PR

[Bug c++/61543] static_cast(static_cast(enum_value)) doesn't get an error

2017-05-11 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61543 Martin Sebor changed: What|Removed |Added Keywords||accepts-invalid

[Bug target/80718] New: GCC generates slow code for offsettable vec_duplicate

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80718 Bug ID: 80718 Summary: GCC generates slow code for offsettable vec_duplicate Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal Priority: P3

[PATCH, ARM/AArch64] drop aarch32 support for falkor/qdf24xx

2017-05-11 Thread Jim Wilson
Early steppings had aarch32 support, current steppings don't, so the aarch32 support for falkor/qdf24xx needs to be dropped. This mostly involves removing falkor/qdf24xx references from the arm port. The qdf24xx_extra_costs structure moves from the arm port to the aarch64 port. This was tested

Re: [PATCH] warn on mem calls modifying objects of non-trivial types (PR 80560)

2017-05-11 Thread Martin Sebor
I found a few more instances of the warning and a couple of bugs in the fixes for it. The attached update corrects those as well. On 05/11/2017 02:01 PM, Martin Sebor wrote: Attached is revision 2 of the patch that addresses of the feedback I got from Pedro (thanks again). As I explained in

[Bug fortran/80333] Namelist dtio write of array of class does not traverse the array

2017-05-11 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80333 Jerry DeLisle changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jvdelisle at gcc dot gnu.org

libgo patch committed: fix signal aliases

2017-05-11 Thread Ian Lance Taylor
This patch by James Cowgill fixes signal aliases when libgo works out the signal table. On MIPS, SIGABRT is defined like this: #define SIGIOT 6 #define SIGABRT SIGIOT This confused the mksigtab.sh script in libgo. This patch fixes the problem. Bootstrapped and ran Go testsuite on

[Bug lto/80717] New: LTO wrappers segfault if run with absolute path

2017-05-11 Thread theivorytower at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80717 Bug ID: 80717 Summary: LTO wrappers segfault if run with absolute path Product: gcc Version: 7.1.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

libgo patch committed: Use opendir64 and closedir64 on AIX

2017-05-11 Thread Ian Lance Taylor
Since on AIX we use readdir64, we must also use opendir64 and closedir64. This patch by Matthieu Sarter fixes the problem. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 06:52 PM, Bernhard Reutner-Fischer wrote: On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: Anyway, please also adjust the dump cleanup to handle class or I can take care of it sometimes later. Applied this, which seems to do the trick. nathan -- Nathan

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 06:57 PM, Rainer Orth wrote: er, the introduction of scanlang.exp broke libatomic, libgomp, and libitm testing. Their logfiles show trunk/12-gcc/build/sparc-sun-solaris2.12/libatomic/testsuite/libatomic.log:ERROR: Couldn't find library file scanlang.exp.

[Bug target/80697] On PowerPC, the spec 2006 benchmark milc had a 5.6% regression under GCC 7.1 compared to GCC 6.3.

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80697 --- Comment #8 from Michael Meissner --- I forgot to mention that I ran the MILC benchmark on a little endian power8 system, using -mcpu=power8 in addition to -O3 and -ffast-math.

[Bug middle-end/80707] [8 Regression] r247844 causes error: extra outgoing edge

2017-05-11 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80707 --- Comment #4 from Peter Bergner --- David and HJ, the following patch fixes the ICE Markus reported, so can you try the following patch to see if it fixes your bootstrap issues? Index: tree-cfg.c

[Bug target/80697] On PowerPC, the spec 2006 benchmark milc had a 5.6% regression under GCC 7.1 compared to GCC 6.3.

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80697 --- Comment #7 from Michael Meissner --- Author: meissner Date: Fri May 12 00:05:24 2017 New Revision: 247941 URL: https://gcc.gnu.org/viewcvs?rev=247941=gcc=rev Log: Attempt to fix pr 80697 Modified:

[Bug target/80697] On PowerPC, the spec 2006 benchmark milc had a 5.6% regression under GCC 7.1 compared to GCC 6.3.

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80697 --- Comment #6 from Michael Meissner --- Author: meissner Date: Fri May 12 00:00:50 2017 New Revision: 247940 URL: https://gcc.gnu.org/viewcvs?rev=247940=gcc=rev Log: Attempt to fix pr 80697 Modified:

[Bug go/64238] ICE in get_partitioning_class, at symtab.c:1775

2017-05-11 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64238 Ian Lance Taylor changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug go/64238] ICE in get_partitioning_class, at symtab.c:1775

2017-05-11 Thread ian at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64238 --- Comment #11 from ian at gcc dot gnu.org --- Author: ian Date: Thu May 11 23:53:29 2017 New Revision: 247939 URL: https://gcc.gnu.org/viewcvs?rev=247939=gcc=rev Log: PR go/64238 * go-gcc.cc

[Bug go/64238] ICE in get_partitioning_class, at symtab.c:1775

2017-05-11 Thread ian at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64238 --- Comment #10 from ian at gcc dot gnu.org --- Author: ian Date: Thu May 11 23:53:13 2017 New Revision: 247938 URL: https://gcc.gnu.org/viewcvs?rev=247938=gcc=rev Log: PR go/64238 * go-gcc.cc

Go patch committed: Fix flag settings for implicit variable reference

2017-05-11 Thread Ian Lance Taylor
The method implicit_variable_reference in the interface used between the Go frontend and GCC proper creates a reference to a variable in a different object file. Unfortunately, ever since it was introduced, it has incorrectly cleared DECL_EXTERNAL and set TREE_STATIC. I'm not entirely sure why

[Bug go/64238] ICE in get_partitioning_class, at symtab.c:1775

2017-05-11 Thread ian at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64238 --- Comment #9 from ian at gcc dot gnu.org --- Author: ian Date: Thu May 11 23:52:54 2017 New Revision: 247937 URL: https://gcc.gnu.org/viewcvs?rev=247937=gcc=rev Log: PR go/64238 * go-gcc.cc

[Bug c/7654] warn if an enum is being assigned a non enum value

2017-05-11 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=7654 Martin Sebor changed: What|Removed |Added Status|SUSPENDED |NEW CC|

Re: [PATCH, v4] Fix PR51513, switch statement with default case containing __builtin_unreachable leads to wild branch

2017-05-11 Thread Peter Bergner
On 5/11/17 3:49 PM, H.J. Lu wrote: > This caused: > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80714 This is already being tracked in https://gcc.gnu.org/PR80707 Peter

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Bernhard Reutner-Fischer
On 12 May 2017 00:57:58 CEST, Rainer Orth wrote: >Bernhard Reutner-Fischer writes: > >> On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: >>>On 05/11/2017 08:28 AM, Rainer Orth wrote: >>> On the gcc-7 branch, the dump

[Bug c++/52763] Warning if compare between enum and non-enum type

2017-05-11 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52763 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org --- Comment #8

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Rainer Orth
Bernhard Reutner-Fischer writes: > On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: >>On 05/11/2017 08:28 AM, Rainer Orth wrote: >> >>> On the gcc-7 branch, the dump file (from -fdump-class-hierarchy) was >>> called covariant7.C.002t.class, now it's

[Bug bootstrap/80714] [8 Regression] profiledbootstrap failed with LTO

2017-05-11 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80714 Peter Bergner changed: What|Removed |Added Status|RESOLVED|CLOSED --- Comment #2 from Peter

[Bug middle-end/80707] [8 Regression] r247844 causes error: extra outgoing edge

2017-05-11 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80707 Peter Bergner changed: What|Removed |Added CC||hjl.tools at gmail dot com --- Comment

[Bug bootstrap/80714] [8 Regression] profiledbootstrap failed with LTO

2017-05-11 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80714 Peter Bergner changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED CC|

Re: [PATCH] make RTL/TREE/IPA dump kind an index

2017-05-11 Thread Bernhard Reutner-Fischer
On 11 May 2017 15:10:36 CEST, Nathan Sidwell wrote: >On 05/11/2017 08:28 AM, Rainer Orth wrote: > >> On the gcc-7 branch, the dump file (from -fdump-class-hierarchy) was >> called covariant7.C.002t.class, now it's covariant7.C.002l.class. >But >> gcc/testsuite/lib/scantree.exp

gcc-7-20170511 is now available

2017-05-11 Thread gccadmin
Snapshot gcc-7-20170511 is now available on ftp://gcc.gnu.org/pub/gcc/snapshots/7-20170511/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 7 SVN branch with the following options: svn://gcc.gnu.org/svn/gcc/branches/gcc-7

[Bug target/80697] On PowerPC, the spec 2006 benchmark milc had a 5.6% regression under GCC 7.1 compared to GCC 6.3.

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80697 --- Comment #5 from Michael Meissner --- Created attachment 41344 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41344=edit Assembler output from gcc 7.1.0

[Bug target/80697] On PowerPC, the spec 2006 benchmark milc had a 5.6% regression under GCC 7.1 compared to GCC 6.3.

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80697 --- Comment #4 from Michael Meissner --- Created attachment 41343 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41343=edit Assembler output from gcc 6.3.0

[Bug target/80697] On PowerPC, the spec 2006 benchmark milc had a 5.6% regression under GCC 7.1 compared to GCC 6.3.

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80697 --- Comment #3 from Michael Meissner --- Created attachment 41342 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41342=edit Preprocessed C source for the mult_adj_su3_mat_vec function To reproduce, you need to use the -O3 -ffast-math

[Bug c++/80716] Death by Documentation (SEGFAULT)

2017-05-11 Thread breakpoint at runbox dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80716 --- Comment #1 from Matt Heck --- Oh, I almost forgot the important part: Substituting commas for those backslashes DOES resolve the problem, but ALL THREE must be replaced. Substituting \\ for \ DOES NOT resolve the problem. Substituting \t

[Bug target/80697] On PowerPC, the spec 2006 benchmark milc had a 5.6% regression under GCC 7.1 compared to GCC 6.3.

2017-05-11 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80697 Michael Meissner changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[patch, libfortran] Fix amount of memory allocation for matrix - vector calculation

2017-05-11 Thread Thomas Koenig
Hello world, the memory allocation for the buffer in the library matmul routines still has one problem: The value of 0xdeadbeef meant as poison could end up in the calculation of the size of the buffer for the blocked matmul. The attached patch fixes that. Verified with regression-test, also by

[Bug c++/80716] New: Death by Documentation (SEGFAULT)

2017-05-11 Thread breakpoint at runbox dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80716 Bug ID: 80716 Summary: Death by Documentation (SEGFAULT) Product: gcc Version: 6.3.1 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++

Re: [PATCH, rs6000] Fold vector logicals in GIMPLE

2017-05-11 Thread Segher Boessenkool
On Thu, May 11, 2017 at 02:36:26PM -0500, Will Schmidt wrote: > On Thu, 2017-05-11 at 14:15 -0500, Segher Boessenkool wrote: > > Hi! > > > > On Thu, May 11, 2017 at 10:53:33AM -0500, Will Schmidt wrote: > > > Add handling for early expansion of vector locical operations in gimple. > > >

[Bug fortran/80666] character length parameter fails if declaration order incorrect

2017-05-11 Thread anlauf at gmx dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80666 Harald Anlauf changed: What|Removed |Added CC||anlauf at gmx dot de --- Comment #3

Re: [RFC PATCH, i386]: Enable post-reload compare elimination pass

2017-05-11 Thread Hans-Peter Nilsson
On Wed, 10 May 2017, Jakub Jelinek wrote: > On Wed, May 10, 2017 at 09:57:56PM +0200, Uros Bizjak wrote: > > BTW: This patch now catches 417 cases (instead of 200+) in linux > > build, including e.g.: > > > > (parallel [ > > (set (reg:CCZ 17 flags) > > (compare:CCZ

[Bug target/79634] ICE in expand_builtin_with_bounds, at builtins.c:7490

2017-05-11 Thread aivchenk at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79634 --- Comment #5 from Alexander Ivchenko --- (In reply to Ilya Enkovich from comment #4) > We hit function pointer propagation issues before and chkp_redirect_edge > with corresponding code in redirect_call_stmt_to_callee appeared to handle >

Re: Go patches committed: merge recent changes to gofrontend

2017-05-11 Thread Rainer Orth
Uros Bizjak writes: >> I have committed a large patch to update the Go frontend and libgo to >> the recent changes in the gofrontend repository. I had postponed >> merging changes during the GCC 7 release process. I am now merging >> all the changes that were pending during

libgo patch committed: fix isSystemGoroutine

2017-05-11 Thread Ian Lance Taylor
The gc toolchain decides whether a goroutine is a system goroutine by comparing startpc to a list of saved special PCs. In gccgo that approach does not work as startpc is often a thunk that invokes the real function with arguments, so the thunk address never matches the saved special PCs. This

[Bug middle-end/80715] New: NULL pointer dereferenced in find_costs_and_classes, at ira-costs.c

2017-05-11 Thread zeccav at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80715 Bug ID: 80715 Summary: NULL pointer dereferenced in find_costs_and_classes, at ira-costs.c Product: gcc Version: 7.1.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH, v4] Fix PR51513, switch statement with default case containing __builtin_unreachable leads to wild branch

2017-05-11 Thread H.J. Lu
On Tue, May 9, 2017 at 7:54 PM, Peter Bergner wrote: > Here is the updated patch to use gimple_seq_unreachable_p() which scans the > sequence backwards so it bails out earlier in the common case (ie, no call > to __builtin_unreachable). As discussed in the previous thread,

[Bug bootstrap/80714] New: [8 Regression] profiledbootstrap failed with LTO

2017-05-11 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80714 Bug ID: 80714 Summary: [8 Regression] profiledbootstrap failed with LTO Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Re: [PATCH, rs6000] gcc mainline, add builtin support for vec_popcnt() and vec_slo()

2017-05-11 Thread Segher Boessenkool
Hi! On Thu, May 11, 2017 at 10:56:48AM -0700, Carl E. Love wrote: > * config/rs6000/rs6000-c: Add support for built-in functions > vector unsigned char vec_popcnt (vector signed char) > vector unsigned char vec_popcnt (vector unsigned char) > vector unsigned short

[Bug fortran/78659] [F03] Spurious "requires DTIO" reported against namelist statement

2017-05-11 Thread jvdelisle at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78659 --- Comment #14 from Jerry DeLisle --- Author: jvdelisle Date: Thu May 11 20:40:49 2017 New Revision: 247930 URL: https://gcc.gnu.org/viewcvs?rev=247930=gcc=rev Log: 2017-05-11 Jerry DeLisle PR fortran/78659

[Bug target/80713] recent crash in update_dep_bb

2017-05-11 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80713 David Binderman changed: What|Removed |Added CC||rguenth at gcc dot gnu.org ---

[Bug target/80713] recent crash in update_dep_bb

2017-05-11 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80713 --- Comment #1 from David Binderman --- Reduced code is this: a, b, d, e, f; *c; g() { for (;;) { if (*c) { int h; *c = } else b = 0; if (f) *c = g; else for (; a; a++) for (;;) {

Re: Bump version namespace and remove _Rb_tree useless template parameter

2017-05-11 Thread François Dumont
On 10/05/2017 22:44, François Dumont wrote: On 10/05/2017 11:28, Jonathan Wakely wrote: On 10/05/17 11:15 +0200, Paolo Carlini wrote: Hi, On 10/05/2017 11:12, Jonathan Wakely wrote: Looks good to me. Paolo, what do you think about bumping the versioned namespace and SONAME to 8? Sure, makes

[Bug target/80695] gratuitous use of stxvx to store multiple pointers

2017-05-11 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80695 Bill Schmidt changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug target/80695] gratuitous use of stxvx to store multiple pointers

2017-05-11 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80695 --- Comment #4 from Bill Schmidt --- Author: wschmidt Date: Thu May 11 20:16:02 2017 New Revision: 247928 URL: https://gcc.gnu.org/viewcvs?rev=247928=gcc=rev Log: [gcc] 2017-05-11 Bill Schmidt PR

[Bug c/80713] New: recent crash in update_dep_bb

2017-05-11 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80713 Bug ID: 80713 Summary: recent crash in update_dep_bb Product: gcc Version: 7.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c

Avoid _Rb_tree_rotate_[left,right] symbols export

2017-05-11 Thread François Dumont
Hi When versioned namespace is active we can avoid export of _Rb_tree_rotate_[left,right] symbols. I also took the opportunity to put static functions in the anonymous namespace rather than using static. Is this usage of static still planned to be deprecated ? * src/c++98/tree.cc

Remove _Safe_container _IsCxx11AllocatorAware template parameter

2017-05-11 Thread François Dumont
Hi _Safe_container _IsCxx11AllocatorAware template allocator is only used if C++11 Abi is not used so I simplified it. * include/debug/safe_container.h [_GLIBCXX_USE_CXX11_ABI] (_Safe_container<>): Remove _IsCxx11AllocatorAware template parameter. * include/debug/string:

Re: [PATCH] warn on mem calls modifying objects of non-trivial types (PR 80560)

2017-05-11 Thread Martin Sebor
Attached is revision 2 of the patch that addresses of the feedback I got from Pedro (thanks again). As I explained in my response (https://gcc.gnu.org/ml/gcc-patches/2017-05/msg00925.html) it's not entirely practical to rely on the strict interpretation of the C++ definition of type triviality

Re: [PATCH, rs6000] Fix PR80695 (vec_construct cost modeling issue)

2017-05-11 Thread Segher Boessenkool
On Thu, May 11, 2017 at 10:59:41AM -0500, Bill Schmidt wrote: > PR80695 identifies a case (similar to several others we've seen) where SLP > vectorization is too aggressive about vectorizing stores. The problem is > that we undervalue the cost of a vec_construct operation. vec_construct > is the

Remove useless debug mode symbols

2017-05-11 Thread François Dumont
Hi Now that soname has been incremented I can remove several useless debug symbols when versioned namespace is active. * include/debug/formatter.h [_GLIBCXX_INLINE_VERSION] (__gnu_debug::_Error_formatter::_Parameter::_M_print_field): Remove.

Re: [PATCH, rs6000] Fold vector logicals in GIMPLE

2017-05-11 Thread Will Schmidt
On Thu, 2017-05-11 at 14:15 -0500, Segher Boessenkool wrote: > Hi! > > On Thu, May 11, 2017 at 10:53:33AM -0500, Will Schmidt wrote: > > Add handling for early expansion of vector locical operations in gimple. > > Specifically: vec_and, vec_andc, vec_or, vec_xor, vec_orc, vec_nand. > > You also

Re: [PATCH, rs6000] Fold vector divides in GIMPLE

2017-05-11 Thread Segher Boessenkool
On Thu, May 11, 2017 at 10:54:02AM -0500, Will Schmidt wrote: > Add handling for early expansion of vector divide built-ins > > Bootstrapped and tested on powerpc64le-unknown-linux-gnu, and > powerpc-unknown-linux, (p7,p8le,p8be) with no regressions. > > OK for trunk? Looks good, please

Re: [PATCH, rs6000] Add x86 instrinsic headers to GCC PPC64LE taget

2017-05-11 Thread Steven Munroe
On Thu, 2017-05-11 at 09:39 -0500, Segher Boessenkool wrote: > On Wed, May 10, 2017 at 12:59:28PM -0500, Steven Munroe wrote: > > > That is just for the testsuite; I meant what happens if a user tries > > > to use it with an older target (or BE, or 32-bit)? Is there a useful, > > > obvious error

Re: [C++ PATCH] pushdecl and friends

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 02:54 PM, Christophe Lyon wrote: On 11 May 2017 at 20:49, Nathan Sidwell wrote: oh gee. I committed the ChangeLog but not the changed file. just done. I have now ... nathan -- Nathan Sidwell

Re: [PATCH] decl lang hooks

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 01:50 PM, Rainer Orth wrote: Hi Nathan, On 05/08/2017 05:34 PM, Joseph Myers wrote: ^~ make: *** [Makefile:1102: objcp/objcp-lang.o] Error 1 Having learnt how to spell --enable-languages properly today, here's the fix,

Re: [PATCH, rs6000] Fold vector logicals in GIMPLE

2017-05-11 Thread Segher Boessenkool
Hi! On Thu, May 11, 2017 at 10:53:33AM -0500, Will Schmidt wrote: > Add handling for early expansion of vector locical operations in gimple. > Specifically: vec_and, vec_andc, vec_or, vec_xor, vec_orc, vec_nand. You also handle nor (except in the changelog). But what about eqv? Segher

Re: [PATCH, rs6000] Fold vector multiplies in GIMPLE

2017-05-11 Thread Segher Boessenkool
Hi! On Thu, May 11, 2017 at 10:51:54AM -0500, Will Schmidt wrote: > Add support for some of the vector multiply (sp,dp) operations in > the gimple folding. I missed these during an earlier pass that > added the other (mule, mulo) multiply operations. This change is > covered by the existing

Re: [C++ PATCH] pushdecl and friends

2017-05-11 Thread Christophe Lyon
On 11 May 2017 at 20:49, Nathan Sidwell wrote: > On 05/11/2017 02:45 PM, Christophe Lyon wrote: > >> >> `/tmp/3713244_6.tmpdir/aci-gcc-fsf/builds/gcc-fsf-gccsrc/obj-aarch64-none-linux-gnu/gcc1/libcc1' >> >> I suspect it will be an obvious fix? > > > oh gee. I committed the

[Bug target/79634] ICE in expand_builtin_with_bounds, at builtins.c:7490

2017-05-11 Thread ienkovich at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79634 --- Comment #4 from Ilya Enkovich --- We hit function pointer propagation issues before and chkp_redirect_edge with corresponding code in redirect_call_stmt_to_callee appeared to handle them. Why doesn't it work in this case?

Re: [C++ PATCH] pushdecl and friends

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 02:45 PM, Christophe Lyon wrote: `/tmp/3713244_6.tmpdir/aci-gcc-fsf/builds/gcc-fsf-gccsrc/obj-aarch64-none-linux-gnu/gcc1/libcc1' I suspect it will be an obvious fix? oh gee. I committed the ChangeLog but not the changed file. just done. Hopefully soon the churn will be

[Bug ipa/79787] ICE in remove_unreachable_nodes, at ipa.c:469

2017-05-11 Thread ienkovich at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79787 Ilya Enkovich changed: What|Removed |Added CC||ienkovich at gcc dot gnu.org ---

Re: [RFC PATCH, i386]: Enable post-reload compare elimination pass

2017-05-11 Thread Uros Bizjak
On Thu, May 11, 2017 at 8:28 PM, Jeff Law wrote: >> Attached patch changes compare-elim.c order to what i386.md expects. >> >> Thoughts? > > Haven't looked at the patch itself. But I do have the necessary bits to > convert the mn103 port. It was slightly more than just fixing

Re: [C++ PATCH] pushdecl and friends

2017-05-11 Thread Christophe Lyon
Hi Nathan On 11 May 2017 at 19:40, Nathan Sidwell wrote: > This patch reduces the variants of pushdecl by use of default arguments. > gone are the _maybe_friend variants. > Since this commit (r247918), GCC builds fail at least for aarch64/arm. I've noticed this error message:

Re: [RFC PATCH, i386]: Enable post-reload compare elimination pass

2017-05-11 Thread Jakub Jelinek
On Thu, May 11, 2017 at 12:28:56PM -0600, Jeff Law wrote: > On 05/10/2017 01:05 PM, Uros Bizjak wrote: > > On Wed, May 10, 2017 at 5:18 PM, Uros Bizjak wrote: > > > On Wed, May 10, 2017 at 4:27 PM, Jakub Jelinek wrote: > > > > On Tue, May 09, 2017 at

Re: Fix for PR79987

2017-05-11 Thread Ilya Enkovich
2017-05-11 0:05 GMT+03:00 Alexander Ivchenko : > Didn't quite get your point. I though that the idea is to hit this code: > > bounds = chkp_generate_extern_var_bounds (decl); > ... to get the builtin_sizeof call on this variable and hence to get > the size relocation.

Re: [RFC PATCH, i386]: Enable post-reload compare elimination pass

2017-05-11 Thread Jeff Law
On 05/10/2017 01:05 PM, Uros Bizjak wrote: On Wed, May 10, 2017 at 5:18 PM, Uros Bizjak wrote: On Wed, May 10, 2017 at 4:27 PM, Jakub Jelinek wrote: On Tue, May 09, 2017 at 06:06:47PM +0200, Uros Bizjak wrote: Attached patch enables post-reload compare

[Bug target/80706] [7/8 Regression] peephole2 uses uninitialized stack variables on i686

2017-05-11 Thread ubizjak at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80706 --- Comment #15 from Uroš Bizjak --- (In reply to Jakub Jelinek from comment #13) > Created attachment 41340 [details] > gcc8-pr80706.patch > > I came up with this (in addition to #c7). This removes one of the two > useless stores in the

[PATCH, i386]: Fix PR 80706, prevent unwanted peephole2 matching

2017-05-11 Thread Uros Bizjak
Hello! Attached patch prevents unwanted peephole2 matching. With SSE, we can expand atomic load through XMM registers using generic DImode move patterns. It can happen that peephole2 matches unrelated insn sequence, and this way removes random FP store to memory. The patch introduces specialized

[Bug target/80706] [7/8 Regression] peephole2 uses uninitialized stack variables on i686

2017-05-11 Thread uros at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80706 --- Comment #14 from uros at gcc dot gnu.org --- Author: uros Date: Thu May 11 18:12:25 2017 New Revision: 247921 URL: https://gcc.gnu.org/viewcvs?rev=247921=gcc=rev Log: PR target/80706 * config/i386/sync.md (UNSPEC_LDX_ATOMIC):

Re: [PATCH] decl lang hooks

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 01:50 PM, Rainer Orth wrote: however, it breaks bootstrap with --enable-languages=obj-c++: wierd, I thought --enable-languges=all enabled that (and I have seen objc issues pop up during development). Will take another look. nathan -- Nathan Sidwell

[PATCH, rs6000] gcc mainline, add builtin support for vec_popcnt() and vec_slo()

2017-05-11 Thread Carl E. Love
GCC Maintainers: This patch adds support for the various vec_popcnt() and vec_slo() builtins. The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) with no regressions. Is the patch OK for gcc mainline? Carl Love

[Bug c++/80712] mention the name of the attribute being ignored

2017-05-11 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80712 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org See

Re: [PATCH] decl lang hooks

2017-05-11 Thread Rainer Orth
Hi Nathan, > On 05/08/2017 05:34 PM, Joseph Myers wrote: >> On Mon, 8 May 2017, Nathan Sidwell wrote: > >>> This patch changes the C++ FE to override the pushdecl and getdecl lang >>> hooks. >>> In addition to simply overriding them there, I had to fixup a couple of >>> places >>> in

Re: [PATCH] Kill -fdump-translation-unit

2017-05-11 Thread Rainer Orth
Hi Nathan, > On 05/11/2017 01:33 PM, Rainer Orth wrote: > >> unfortunately, this part >> >> * graphite-poly.c: Don't include tree-dump.h. >> >> broke bootstrap with graphite: > > oh, that has to be configured enabled? I've committed this that I think is right: it requires an appropriate

Re: [PATCH] Kill -fdump-translation-unit

2017-05-11 Thread Nathan Sidwell
On 05/11/2017 01:33 PM, Rainer Orth wrote: unfortunately, this part * graphite-poly.c: Don't include tree-dump.h. broke bootstrap with graphite: oh, that has to be configured enabled? I've committed this that I think is obvious. looks to be another case of using tree-dumph to get

[C++ PATCH] pushdecl and friends

2017-05-11 Thread Nathan Sidwell
This patch reduces the variants of pushdecl by use of default arguments. gone are the _maybe_friend variants. nathan -- Nathan Sidwell 2017-05-11 Nathan Sidwell * name-lookup.h (pushdecl): Add default friend parm. (pushdecl_maybe_friend): Delete. (pushdecl_top_level):

Re: [PATCH, rs6000] Fix PR80695 (vec_construct cost modeling issue)

2017-05-11 Thread Bill Schmidt
I meant to mention that I regstrapped this on both POWER8 and POWER9. Bill > On May 11, 2017, at 10:59 AM, Bill Schmidt > wrote: > > Hi, > > PR80695 identifies a case (similar to several others we've seen) where SLP > vectorization is too aggressive about

Re: [PATCH] Kill -fdump-translation-unit

2017-05-11 Thread Rainer Orth
Hi Nathan, > On 05/10/2017 04:49 AM, Richard Biener wrote: >> On Tue, May 9, 2017 at 5:41 PM, Nathan Sidwell wrote: >>> -fdump-translation-unit is an inscrutably opaque dump. It turned out that >>> most of the uses of the tree-dump header file was to indirectly get at >>>

Re: Separate preprocess and compile: hack or feature?

2017-05-11 Thread Boris Kolpackov
Hi Nathan, Nathan Sidwell writes: > How c++ modules fit into a build system is currently an open question. > Richard Smith & I have talked about it, but with no firm conclusion. > However, I think that breaking out the preprocessor is not the right > answer. Handling modules is

Re: [PATCH] Move target independent code to support target_clones attributes from i386 to common code

2017-05-11 Thread Jeff Law
On 05/10/2017 04:40 PM, Michael Meissner wrote: As I mentioned in the mail message: https://gcc.gnu.org/ml/gcc/2017-05/msg00060.html I'm working on adding the target_clones attribute support to the PowerPC. I have an implementation right now, but I want to iterate on it somewhat. In doing the

[Bug c++/80712] mention the name of the attribute being ignored

2017-05-11 Thread glisse at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80712 Marc Glisse changed: What|Removed |Added Keywords||diagnostic Status|UNCONFIRMED

Re: PATCH ATTACHED Re: [PATCH, Fortran] PR78659 Spurious "requires DTIO" reported against namelist statement

2017-05-11 Thread Paul Richard Thomas
Hi Jerry, This patch is good for both trunk and 7-branch. Thanks! Paul On 11 May 2017 at 16:35, Jerry DeLisle wrote: > And the actual patch ... > > On 05/11/2017 08:30 AM, Jerry DeLisle wrote: >> >> Hi all, >> >> The attached patch fixes this issue by moving the DTIO

Re: [RFC PATCH, i386]: Enable post-reload compare elimination pass

2017-05-11 Thread Jeff Law
On 05/10/2017 02:27 PM, Jakub Jelinek wrote: On Wed, May 10, 2017 at 09:57:56PM +0200, Uros Bizjak wrote: BTW: This patch now catches 417 cases (instead of 200+) in linux build, including e.g.: (parallel [ (set (reg:CCZ 17 flags) (compare:CCZ (lshiftrt:SI (reg:SI 4 si

[Bug c++/70538] ICE on a deprecated access declaration in instantiate_template_1, at cp/pt.c:17310

2017-05-11 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70538 Paolo Carlini changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug c++/70538] ICE on a deprecated access declaration in instantiate_template_1, at cp/pt.c:17310

2017-05-11 Thread paolo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70538 --- Comment #5 from paolo at gcc dot gnu.org --- Author: paolo Date: Thu May 11 17:13:42 2017 New Revision: 247917 URL: https://gcc.gnu.org/viewcvs?rev=247917=gcc=rev Log: 2017-05-11 Paolo Carlini PR

[Bug middle-end/80707] [8 Regression] r247844 causes error: extra outgoing edge

2017-05-11 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80707 David Binderman changed: What|Removed |Added CC||dcb314 at hotmail dot com --- Comment

[Bug c++/70538] ICE on a deprecated access declaration in instantiate_template_1, at cp/pt.c:17310

2017-05-11 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70538 --- Comment #4 from Paolo Carlini --- This is fixed in 7.1.0: I'm adding a testcase and closing the bug.

  1   2   3   >