Re: Extend aligned_membuf<> usage

2018-02-07 Thread François Dumont
On 06/02/2018 20:16, François Dumont wrote: On 05/02/2018 18:16, Jonathan Wakely wrote: Wouldn't it make more sense to simply make __aligned_buffer identical to __aligned_membuf for the versioned-namespace? Then at least the conditional code is only in one place. Yes, __aligned_buffer is

[PATCH] Fix -fcompare-debug failure on pr84146.c (PR target/84146)

2018-02-07 Thread Jakub Jelinek
Hi! Unfortunately, seems my rest_of_insert_endbranch fix doesn't fix -fcompare-debug on the testcase, when adding the endbr after the setjmp call with no note in between, we add it into the same bb as the setjmp call, while when adding it with -g with NOTE_INSN_CALL_ARG_LOCATION, which is already

Re: PATCH to fix bogus warning with -Wstringop-truncation -g (PR tree-optimization/84228)

2018-02-07 Thread Jeff Law
On 02/06/2018 08:59 AM, Martin Sebor wrote: > On 02/06/2018 06:23 AM, Marek Polacek wrote: >> On Tue, Feb 06, 2018 at 01:57:36PM +0100, Jakub Jelinek wrote: >>> On Tue, Feb 06, 2018 at 01:46:21PM +0100, Marek Polacek wrote: --- gcc/testsuite/c-c++-common/Wstringop-truncation-3.c +++

Re: PATCH to fix bogus warning with -Wstringop-truncation -g (PR tree-optimization/84228)

2018-02-07 Thread Jeff Law
On 02/06/2018 05:57 AM, Jakub Jelinek wrote: > On Tue, Feb 06, 2018 at 01:46:21PM +0100, Marek Polacek wrote: >> --- gcc/testsuite/c-c++-common/Wstringop-truncation-3.c >> +++ gcc/testsuite/c-c++-common/Wstringop-truncation-3.c >> @@ -0,0 +1,20 @@ >> +/* PR tree-optimization/84228 */ >> +/* {

Re: PATCH to fix bogus warning with -Wstringop-truncation -g (PR tree-optimization/84228)

2018-02-07 Thread Jeff Law
On 02/06/2018 05:46 AM, Marek Polacek wrote: > When -Wstringop-truncation sees a strncpy call where the specified bound > is equal to the size of the destination, it looks at the next statement > to see if it's dst[i] = '\0';, and if it is, it doesn't warn. But it > needs to look at the next

Re: [PATCH/RFC] Fix ICE in find_taken_edge_computed_goto (PR 84136)

2018-02-07 Thread Jeff Law
On 02/02/2018 02:35 PM, David Malcolm wrote: > On Thu, 2018-02-01 at 12:05 +0100, Richard Biener wrote: >> On Wed, Jan 31, 2018 at 4:39 PM, David Malcolm >> wrote: >>> PR 84136 reports an ICE within sccvn_dom_walker when handling a >>> C/C++ source file that overuses the

Re: [PATCH/RFC] Fix ICE in find_taken_edge_computed_goto (PR 84136)

2018-02-07 Thread Jeff Law
On 02/01/2018 04:05 AM, Richard Biener wrote: > On Wed, Jan 31, 2018 at 4:39 PM, David Malcolm wrote: >> PR 84136 reports an ICE within sccvn_dom_walker when handling a >> C/C++ source file that overuses the labels-as-values extension. >> The code in question stores a jump

Re: [PATCH/RFC] Fix ICE in find_taken_edge_computed_goto (PR 84136)

2018-02-07 Thread Jeff Law
On 01/31/2018 08:39 AM, David Malcolm wrote: > PR 84136 reports an ICE within sccvn_dom_walker when handling a > C/C++ source file that overuses the labels-as-values extension. > The code in question stores a jump label into a global, and then > jumps to it from another function, which ICEs after

Re: [PR tree-optimization/84224] do not ICE on malformed allocas

2018-02-07 Thread Jeff Law
On 02/06/2018 02:38 AM, Aldy Hernandez wrote: > The -Walloca pass can receive a malformed alloca, courtesy of someone > providing a faulty prototype.  This was causing an ICE because we > assumed alloca calls had at least one argument, which the testcase does > not: > > +void *alloca (); >

Re: [PATCH] Fix floating point handling in dom (PR tree-optimization/84235)

2018-02-07 Thread Jeff Law
On 02/07/2018 12:06 AM, Richard Biener wrote: > On February 6, 2018 9:40:37 PM GMT+01:00, Jakub Jelinek > wrote: >> Hi! >> >> As the following testcase shows, dom2 miscompiles floating point x - x >> into 0.0 even when x could be infinity and x - x then a NaN. >> The

[Bug target/84266] mmintrin.h intrinsic headers for PowerPC code fails on power9

2018-02-07 Thread seurer at linux dot vnet.ibm.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84266 --- Comment #3 from Bill Seurer --- All the compilation failures disappear if the two casts /home/seurer/gcc/build/gcc-trunk/gcc/include/mmintrin.h:857:5: error: incompatible types when assigning to type '__vector signed int' {aka '__vector(4)

[Bug fortran/84276] Invalid error for valid statement function

2018-02-07 Thread sgk at troutmask dot apl.washington.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84276 --- Comment #2 from Steve Kargl --- On Thu, Feb 08, 2018 at 02:26:50AM +, sgk at troutmask dot apl.washington.edu wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84276 > > --- Comment #1 from Steve Kargl --- > Reduced testcase. > >

[Bug c++/82930] [C++17] ICE: in is_normal_capture_proxy, at cp/lambda.c:288 with structured binding in a lambda function with auto typed arguments

2018-02-07 Thread hotguest1 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82930 sandthorn changed: What|Removed |Added CC||hotguest1 at hotmail dot com --- Comment #3

Re: [PATCH] add -Wstringop-overflow to LTO options (PR 84212)

2018-02-07 Thread Martin Sebor
I went ahead and changed all the options on the list below to include LTO and tested the attached patch by configuring with --with-build-config=bootstrap-lto --disable-werror and making profiledbootstrap. Attached, besides the patch, is also the breakdown of warnings. The interesting column is

[Bug fortran/84276] Invalid error for valid statement function

2018-02-07 Thread sgk at troutmask dot apl.washington.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84276 --- Comment #1 from Steve Kargl --- Reduced testcase. subroutine stepns(hh,h,s,w) real, intent(inout) :: h,hh,s real, intent(out) :: w real :: qofs qofs(s)=s w=qofs(hh+h) end subroutine stepns Problem

[Bug c/84258] Bogus -Wformat warning "wide character string" for const unsigned char

2018-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84258 David Malcolm changed: What|Removed |Added Status|ASSIGNED|WAITING --- Comment #4 from David

[Bug fortran/84276] Invalid error for valid statement function

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84276 kargl at gcc dot gnu.org changed: What|Removed |Added Priority|P3 |P4 Known to fail|

[PATCH] -Wformat: fix nonsensical "wide character" message (PR c/84258)

2018-02-07 Thread David Malcolm
PR c/84258 reports that we issue this: warning: format is a wide character string [-Wformat=] on this code: const unsigned char cuc[] = "%i"; sprintf(buf, (char *)cuc, 1); despite the absence of wide characters. This wording dates back 17.5 years to r36586: 2000-09-24 Joseph S. Myers

[Bug fortran/84276] New: Invalid error for valid statement function

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84276 Bug ID: 84276 Summary: Invalid error for valid statement function Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: fortran

[Bug middle-end/84275] New: missing warning on conflicting attributes on different declara

2018-02-07 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84275 Bug ID: 84275 Summary: missing warning on conflicting attributes on different declara Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal

Re: gcc 7.3: Replacing global operator new/delete in shared libraries

2018-02-07 Thread Paul Smith
On Thu, 2018-02-08 at 01:17 +0100, Marc Glisse wrote: > On Wed, 7 Feb 2018, Paul Smith wrote: > > > > My question is, what do I need to do to ensure this behavior > > > > persists if I create a global operator new/delete? > > > > > > > > Is it sufficient to ensure that the symbol for our shared >

Re: [RFC][PR82479] missing popcount builtin detection

2018-02-07 Thread Kugan Vivekanandarajah
Hi Richard, On 1 February 2018 at 23:21, Richard Biener wrote: > On Thu, Feb 1, 2018 at 5:07 AM, Kugan Vivekanandarajah > wrote: >> Hi Richard, >> >> On 31 January 2018 at 21:39, Richard Biener >>

Re: gcc 7.3: Replacing global operator new/delete in shared libraries

2018-02-07 Thread Paul Smith
On Wed, 2018-02-07 at 16:38 -0700, Martin Sebor wrote: > I'm not sure I see how defining operator new inline can work > unless you recompile the world (i.e., all the DSOs used by > the program, including libstdc++). As Marc already hinted, > if libstdc++ dynamically allocates an object using the

[Bug c/84274] New: [feature request] mbind attribute

2018-02-07 Thread jeff.science at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84274 Bug ID: 84274 Summary: [feature request] mbind attribute Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c

Re: gcc 7.3: Replacing global operator new/delete in shared libraries

2018-02-07 Thread Marc Glisse
On Wed, 7 Feb 2018, Paul Smith wrote: My question is, what do I need to do to ensure this behavior persists if I create a global operator new/delete? Is it sufficient to ensure that the symbol for our shared library global new/delete symbols are hidden and not global, using a linker map or

Re: [PATCH], Document the PowerPC --with-long-double-format={ibm,ieee} configuration options

2018-02-07 Thread Michael Meissner
On Wed, Feb 07, 2018 at 05:26:49PM -0600, Segher Boessenkool wrote: > On Wed, Feb 07, 2018 at 05:48:26PM -0500, Michael Meissner wrote: > > This patch udpates the GCC installation documentation to document the > > configuration options to set the long double type on the PowerPC: I verified > >

Re: gcc 7.3: Replacing global operator new/delete in shared libraries

2018-02-07 Thread Jonathan Wakely
On 7 February 2018 at 23:38, Martin Sebor wrote: > On 02/06/2018 03:56 PM, Paul Smith wrote: >> >> Hi all. >> >> Hopefully this isn't too annoying a question :). >> >> My environment has been using GCC 6.2 (locally compiled) on GNU/Linux >> systems. We use a separate heap management library

Re: Please accept this commit for the trunk

2018-02-07 Thread Mike Stump
On Feb 5, 2018, at 8:42 AM, Douglas Mencken wrote: > > I’m about > > “ [PATCH 2/4] [Darwin,PPC] Remove uses of LR in > restore_world ” https://gcc.gnu.org/bugzilla/attachment.cgi?id=42304 > > look at bug #84113 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84113 for >

Re: gcc 7.3: Replacing global operator new/delete in shared libraries

2018-02-07 Thread Martin Sebor
On 02/06/2018 03:56 PM, Paul Smith wrote: Hi all. Hopefully this isn't too annoying a question :). My environment has been using GCC 6.2 (locally compiled) on GNU/Linux systems. We use a separate heap management library (jemalloc) rather than the libc allocator. The way we did this in the

Re: [PATCH, rs6000] Update vsx-vector-6-le.c tests for p9 target

2018-02-07 Thread Segher Boessenkool
On Wed, Feb 07, 2018 at 05:23:31PM -0600, Will Schmidt wrote: > > > /* { dg-do compile { target { powerpc64le-*-* && lp64 } } } */ > > > /* { dg-skip-if "" { powerpc*-*-darwin* } } */ > > > /* { dg-require-effective-target powerpc_vsx_ok } */ > > > -/* { dg-options "-mvsx -O2" } */ > > > +/* {

Re: [PATCH], Document the PowerPC --with-long-double-format={ibm,ieee} configuration options

2018-02-07 Thread Segher Boessenkool
On Wed, Feb 07, 2018 at 05:48:26PM -0500, Michael Meissner wrote: > This patch udpates the GCC installation documentation to document the > configuration options to set the long double type on the PowerPC: I verified > that the documentation builds. Can I install this on to the trunk? Yes

Re: [PATCH, rs6000] Update vsx-vector-6-le.c tests for p9 target

2018-02-07 Thread Will Schmidt
On Wed, 2018-02-07 at 12:28 -0600, Segher Boessenkool wrote: > Hi! > > On Wed, Feb 07, 2018 at 11:16:12AM -0600, Will Schmidt wrote: > > Noted during review of test results on P9. Due to changes and > > improvements, > > our codegen is different for this test on power9. > > Modified the

[Bug target/84154] [7/8 Regression] PowerPC GCC 7 and 8 have regression in converting fp to short/char and returning it

2018-02-07 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84154 Michael Meissner changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

Re: GCC 8.0.0 status on x86_64-w64-mingw32, some issues

2018-02-07 Thread Eric Botcazou
> Indeed, this solves most of the new failures. Here is the acats test > summary: > === acats Summary === > # of expected passes 2298 > # of unexpected failures 22 > *** FAILURES: c23003b c23003g c23003i c250002 c380004 cd2b11a cd2b15c > ce2102l ce2102m ce2103a ce2103b

Re: [PATCH, rs6000] fix vsxcopy test for power9

2018-02-07 Thread Will Schmidt
On Wed, 2018-02-07 at 12:02 -0600, Segher Boessenkool wrote: > Hi, > > On Wed, Feb 07, 2018 at 10:59:46AM -0600, Will Schmidt wrote: > > Noted during review of test results on P9. The vsxcopy.c test is looking > > for lxvd2x, stxvd2x instructions in generated code. For P9 targets, we will > >

[Bug target/84154] [7/8 Regression] PowerPC GCC 7 and 8 have regression in converting fp to short/char and returning it

2018-02-07 Thread meissner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84154 --- Comment #3 from Michael Meissner --- Author: meissner Date: Wed Feb 7 22:54:59 2018 New Revision: 257470 URL: https://gcc.gnu.org/viewcvs?rev=257470=gcc=rev Log: [gcc] 2018-02-07 Michael Meissner PR

[PATCH], Document the PowerPC --with-long-double-format={ibm,ieee} configuration options

2018-02-07 Thread Michael Meissner
This patch udpates the GCC installation documentation to document the configuration options to set the long double type on the PowerPC: I verified that the documentation builds. Can I install this on to the trunk? 2018-02-07 Michael Meissner *

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Jakub Jelinek
On Wed, Feb 07, 2018 at 03:52:39PM -0500, Jason Merrill wrote: > > E.g. the constexpr function uses same_type_ignoring_top_level_qualifiers_p > > instead of == type comparisons, the COMPONENT_REF stuff, ... > > > For poly_* stuff, I think Richard S. wants to introduce it into the FEs at > > some

[PATCH] Fix var-tracking ICE introduced in poly_int conversion (PR debug/84252)

2018-02-07 Thread Jakub Jelinek
Hi! As mentioned in the PR, the vt_get_decl_and_offset function verifies incoming PARALLEL is usable for tracking, but if it fails, we retry vt_get_decl_and_offset on DECL_RTL and there we check only that a memory isn't larger than 16 bytes (to make sure it doesn't have more than 16 parts), but

[Bug fortran/82994] ICE in gfc_match_deallocate, at fortran/match.c:4478

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82994 --- Comment #6 from kargl at gcc dot gnu.org --- Author: kargl Date: Wed Feb 7 22:41:59 2018 New Revision: 257468 URL: https://gcc.gnu.org/viewcvs?rev=257468=gcc=rev Log: 2018-02-07 Steven G. Kargl PR

gcc-6-20180207 is now available

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

[Bug fortran/82994] ICE in gfc_match_deallocate, at fortran/match.c:4478

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82994 kargl at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

Re: [PATCH] PowerPC PR target/84154, fix floating point to small integer conversion regression

2018-02-07 Thread Segher Boessenkool
Hi Mike, On Tue, Feb 06, 2018 at 04:34:08PM -0500, Michael Meissner wrote: > Here is the patch reworked. It bootstraps on both little/big endian power8, > and all of the tests run. Can I install this into trunk now, and into GCC 7 > after a soak period (along with the previous patch)? > +;; If

[Bug target/83707] g++.dg/eh/simd-3.C fails on power7 -m32

2018-02-07 Thread willschm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83707 --- Comment #3 from Will Schmidt --- Created attachment 43360 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43360=edit .expand dump from a build with -O1. the .expand dump from a build with -O1.

[Bug fortran/82994] ICE in gfc_match_deallocate, at fortran/match.c:4478

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82994 --- Comment #5 from kargl at gcc dot gnu.org --- Author: kargl Date: Wed Feb 7 22:36:24 2018 New Revision: 257467 URL: https://gcc.gnu.org/viewcvs?rev=257467=gcc=rev Log: 2018-02-07 Steven G. Kargl PR

[Bug target/83707] g++.dg/eh/simd-3.C fails on power7 -m32

2018-02-07 Thread willschm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83707 Will Schmidt changed: What|Removed |Added CC||willschm at gcc dot gnu.org --- Comment

[Bug c++/84082] [7/8 Regression] ICE with broken template function definition

2018-02-07 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84082 --- Comment #6 from Jakub Jelinek --- Author: jakub Date: Wed Feb 7 22:30:51 2018 New Revision: 257466 URL: https://gcc.gnu.org/viewcvs?rev=257466=gcc=rev Log: PR c++/84082 * parser.c (cp_parser_dot_deref_incomplete): New

[Committed] PR fortran/82994 -- Check for a NULL pointer

2018-02-07 Thread Steve Kargl
Committed as obvious. 2018-02-07 Steven G. Kargl PR fortran/82994 * match.c (gfc_match_deallocate): Check for NULL pointer. 2018-02-07 Steven G. Kargl PR fortran/82994 * gfortran.dg/deallocate_error_3.f90: New test.

[Bug fortran/82994] ICE in gfc_match_deallocate, at fortran/match.c:4478

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82994 --- Comment #4 from kargl at gcc dot gnu.org --- Author: kargl Date: Wed Feb 7 22:29:22 2018 New Revision: 257465 URL: https://gcc.gnu.org/viewcvs?rev=257465=gcc=rev Log: 2018-02-07 Steven G. Kargl PR

New Spanish PO file for 'gcc' (version 8.1-b20180128)

2018-02-07 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: http://translationproject.org/latest/gcc/es.po (This file, 'gcc-8.1-b20180128.es.po',

Re: [PATCH] libgcc: xtensa: fix build with -mtext-section-literals

2018-02-07 Thread Max Filippov
On Thu, Feb 1, 2018 at 9:12 AM, Max Filippov wrote: > On Wed, Jan 31, 2018 at 11:17 AM, Max Filippov wrote: >> On Wed, Jan 31, 2018 at 11:11 AM, augustine.sterl...@gmail.com >> wrote: >>> On Tue, Jan 30, 2018 at 8:02 PM, Max

[Bug c++/82478] Rejects valid access to private member type that should be allowed by friend

2018-02-07 Thread zamazan4ik at tut dot by
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82478 Alexander Zaitsev changed: What|Removed |Added CC||zamazan4ik at tut dot by ---

libgo patch committed: Don't call funcPC from a function

2018-02-07 Thread Ian Lance Taylor
This patch changes the libgo runtime package to not call funcPC from a function. The escape analysis support is not yet good enough to avoid escaping the argument to funcPC. This causes unnecessary and often harmful memory allocation. E.g., (*cpuProfile).addExtra can be called from a signal

Re: gdb 8.x - g++ 7.x compatibility

2018-02-07 Thread Nathan Sidwell
On 02/07/2018 12:11 PM, Daniel Berlin wrote: Note that the ABI is explicitly designed so that type identity can be done by address comparison. correct, but be aware that lots of dynamic objects seem to step outside the ABI by building shared objects with -Bsymbolic[1], or the equivalent

[Bug fortran/84273] New: Reject allocatable passed-object dummy argument (proc_ptr_47.f90)

2018-02-07 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84273 Bug ID: 84273 Summary: Reject allocatable passed-object dummy argument (proc_ptr_47.f90) Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Jakub Jelinek
On Wed, Feb 07, 2018 at 08:36:31PM +0100, Marek Polacek wrote: > > > That was my first patch, but it was rejected: > > > https://gcc.gnu.org/ml/gcc-patches/2018-01/msg00271.html > > > > Then should we update fold_indirect_ref_1 to use the new code? Is > > there a reason for them to stay out of

[Bug c/83390] valgrind error in lra_eliminate_regs_1

2018-02-07 Thread vmakarov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83390 --- Comment #8 from Vladimir Makarov --- (In reply to David Binderman from comment #0) > A build of today's gcc trunk with valgrind produces this: > > ==8995== Conditional jump or move depends on uninitialised value(s) > ==8995==at

Re: [C++ Patch/RFC] PR 83204 ("[6/7/8 Regression] c++ -std=c++14 ICE in maybe_undo_parenthesized_ref, at cp/semantics.c:1694")

2018-02-07 Thread Jason Merrill
OK. On Wed, Feb 7, 2018 at 4:16 PM, Paolo Carlini wrote: > Hi, > > this is essentially an RFC, I'm still fully analyzing the issue, but since I > already have something prima facie reasonable and passing the testsuite I > decided to send immediately out what I have,

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Jason Merrill
On Wed, Feb 7, 2018 at 4:14 PM, Jakub Jelinek wrote: > On Wed, Feb 07, 2018 at 03:52:39PM -0500, Jason Merrill wrote: >> > E.g. the constexpr function uses same_type_ignoring_top_level_qualifiers_p >> > instead of == type comparisons, the COMPONENT_REF stuff, ... >> >> > For

[Bug fortran/84218] ICE in free_expr0, at fortran/expr.c:451

2018-02-07 Thread neil.n.carlson at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84218 Neil Carlson changed: What|Removed |Added CC||neil.n.carlson at gmail dot com ---

[C++ Patch/RFC] PR 83204 ("[6/7/8 Regression] c++ -std=c++14 ICE in maybe_undo_parenthesized_ref, at cp/semantics.c:1694")

2018-02-07 Thread Paolo Carlini
Hi, this is essentially an RFC, I'm still fully analyzing the issue, but since I already have something prima facie reasonable and passing the testsuite I decided to send immediately out what I have, looking for further feedback / help. As fully analyzed by Jakub in the audit trail, for the

[Bug fortran/84270] optimization bug with assumed size array argument

2018-02-07 Thread anlauf at gmx dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84270 Harald Anlauf changed: What|Removed |Added CC||anlauf at gmx dot de --- Comment #1

[Bug target/84272] AddressSanitizer: heap-use-after-free ../../gcc/config/aarch64/cortex-a57-fma-steering.c:519 in fma_node::get_parity()

2018-02-07 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84272 Martin Liška changed: What|Removed |Added Priority|P3 |P1 Last reconfirmed|

[Bug target/84272] New: AddressSanitizer: heap-use-after-free ../../gcc/config/aarch64/cortex-a57-fma-steering.c:519 in fma_node::get_parity()

2018-02-07 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84272 Bug ID: 84272 Summary: AddressSanitizer: heap-use-after-free ../../gcc/config/aarch64/cortex-a57-fma-steering.c:519 in fma_node::get_parity() Product: gcc

[Bug fortran/68560] [6/7/8 Regression] The test gfortran.dg/shape_8.f90 now fails when compiled with -flto

2018-02-07 Thread tkoenig at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68560 --- Comment #28 from Thomas Koenig --- Author: tkoenig Date: Wed Feb 7 21:08:51 2018 New Revision: 257462 URL: https://gcc.gnu.org/viewcvs?rev=257462=gcc=rev Log: 2018-02-07 Thomas Koenig PR fortran/68560

[Bug fortran/82049] [6/7/8 Regression] ICE with character(*),parameter array constructor

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82049 kargl at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

Re: [PATCH] PR fortran/82049 -- resolve a charlen if possible

2018-02-07 Thread Steve Kargl
Thanks. A version of the patch has been commit to 6-branch, 7-branch, and trunk. One regression down, many more to go. -- steve On Wed, Feb 07, 2018 at 08:10:41AM +, Paul Richard Thomas wrote: > Hi Steve, > > That's OK for trunk and, if you are possessed of the intestinal > fortitude, 6-

[Bug fortran/82049] [6/7/8 Regression] ICE with character(*),parameter array constructor

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82049 --- Comment #5 from kargl at gcc dot gnu.org --- Author: kargl Date: Wed Feb 7 21:01:00 2018 New Revision: 257461 URL: https://gcc.gnu.org/viewcvs?rev=257461=gcc=rev Log: 2018-02-07 Steven G. Kargl PR

[Bug target/78303] PowerPC vec-init-{1,2,4,5,8,9} tests do not run on -mlittle -maltivec=be

2018-02-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78303 --- Comment #4 from Segher Boessenkool --- If -maltivec=be is not used, I support removing it. Deprecating it with a warning on use will show if anyone care enough to tell us they do use it ;-)

[Bug c/69558] [6/7/8/9 Regression] glib2 warning pragmas stopped working

2018-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69558 David Malcolm changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Jason Merrill
On Wed, Feb 7, 2018 at 3:32 PM, Jakub Jelinek wrote: > On Wed, Feb 07, 2018 at 03:23:25PM -0500, Jason Merrill wrote: >> On Wed, Feb 7, 2018 at 2:48 PM, Jakub Jelinek wrote: >> > On Wed, Feb 07, 2018 at 08:36:31PM +0100, Marek Polacek wrote: >> >> > > That was

Re: [patch, fortran] Fix PR 68560

2018-02-07 Thread Steve Kargl
On Wed, Feb 07, 2018 at 09:42:04PM +0100, Thomas Koenig wrote: > Here's an update on the patch - I realized that it is not necessary > to check for the actual argument, it is always present. > > OK for trunk? > Yes. -- Steve

[Bug fortran/82049] [6/7/8 Regression] ICE with character(*),parameter array constructor

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82049 --- Comment #4 from kargl at gcc dot gnu.org --- Author: kargl Date: Wed Feb 7 20:45:40 2018 New Revision: 257460 URL: https://gcc.gnu.org/viewcvs?rev=257460=gcc=rev Log: 2018-02-07 Steven G. Kargl PR

[Bug fortran/82049] [6/7/8 Regression] ICE with character(*),parameter array constructor

2018-02-07 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82049 --- Comment #3 from kargl at gcc dot gnu.org --- Author: kargl Date: Wed Feb 7 20:43:33 2018 New Revision: 257459 URL: https://gcc.gnu.org/viewcvs?rev=257459=gcc=rev Log: 2018-02-06 Steven G. Kargl PR

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Jakub Jelinek
On Wed, Feb 07, 2018 at 03:23:25PM -0500, Jason Merrill wrote: > On Wed, Feb 7, 2018 at 2:48 PM, Jakub Jelinek wrote: > > On Wed, Feb 07, 2018 at 08:36:31PM +0100, Marek Polacek wrote: > >> > > That was my first patch, but it was rejected: > >> > >

Re: [patch, fortran] Fix PR 68560

2018-02-07 Thread Thomas Koenig
Here's an update on the patch - I realized that it is not necessary to check for the actual argument, it is always present. OK for trunk? Regards Thomas 2018-02-01  Thomas Koenig  PR fortran/68560 * trans-intrinsic.c (gfc_conv_intrinsic_shape): New

[Bug c++/83204] [6/7/8 Regression] c++ -std=c++14 ICE in maybe_undo_parenthesized_ref, at cp/semantics.c:1694

2018-02-07 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83204 Paolo Carlini changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned

Re: gdb 8.x - g++ 7.x compatibility

2018-02-07 Thread Tom Tromey
> "Dan" == Daniel Berlin writes: Dan> If there are multiple types named Foo<2u>, DWARF needs to be extended to Dan> allow a pointer from the vtable debug info to the class type debug info Dan> (unless they already added one). This is what we did for Rust. Rust doesn't

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Jason Merrill
On Wed, Feb 7, 2018 at 2:48 PM, Jakub Jelinek wrote: > On Wed, Feb 07, 2018 at 08:36:31PM +0100, Marek Polacek wrote: >> > > That was my first patch, but it was rejected: >> > > https://gcc.gnu.org/ml/gcc-patches/2018-01/msg00271.html >> > >> > Then should we update

Re: [openacc, committed] Fix diff_type in expand_oacc_collapse_init

2018-02-07 Thread Tom de Vries
On 02/07/2018 08:37 PM, Rainer Orth wrote: Hi Tom, this patch fixes an 8 regression in an openacc testcase. The regression was introduced by r250925, a fix for PR78266, a bug in the handling of a loop with iteration variable type range smaller than the size of the parallel dimension the loop

[Bug target/84266] mmintrin.h intrinsic headers for PowerPC code fails on power9

2018-02-07 Thread wschmidt at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84266 --- Comment #2 from Bill Schmidt --- I wonder how many failures are left if that invalid cast is removed from the code? It is just wrong and unnecessary.

Re: [RFC PATCH] avoid applying attributes to explicit specializations (PR 83871)

2018-02-07 Thread Jason Merrill
On 02/06/2018 11:01 PM, Martin Sebor wrote: On 02/05/2018 02:52 PM, Jason Merrill wrote: On 02/04/2018 07:07 PM, Martin Sebor wrote: To resolve the underlying root cause of the P1 bug c++/83503 - bogus -Wattributes for const and pure on function template specialization, that we discussed last

Re: [openacc, committed] Fix diff_type in expand_oacc_collapse_init

2018-02-07 Thread Rainer Orth
Hi Tom, > this patch fixes an 8 regression in an openacc testcase. > > The regression was introduced by r250925, a fix for PR78266, a bug in the > handling of a loop with iteration variable type range smaller than the size > of the parallel dimension the loop is assigned to. > > The fix for the

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Marek Polacek
On Wed, Feb 07, 2018 at 01:23:49PM -0500, Jason Merrill wrote: > On Wed, Feb 7, 2018 at 12:54 PM, Marek Polacek wrote: > > On Wed, Feb 07, 2018 at 12:26:04PM -0500, Jason Merrill wrote: > >> On Fri, Jan 26, 2018 at 6:22 PM, Jakub Jelinek wrote: > >> > On

[Bug lto/84212] -Wno-* does not disable warnings from -flto link stage

2018-02-07 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84212 Martin Sebor changed: What|Removed |Added Keywords||patch --- Comment #6 from Martin Sebor

[PATCH] add -Wstringop-overflow to LTO options (PR 84212)

2018-02-07 Thread Martin Sebor
In PR 84212 the reporter asks why -Wno-stringop-overflow has no effect during LTO linking. It turns out that the reason is the same as in bug 78768: the specification in the c.opt file is missing LTO among the languages. The attached patch adds LTO to it and to -Wstringop-truncation.

Re: [SFN+LVU+IEPM v4 7/9] [LVU] Introduce location views

2018-02-07 Thread Jason Merrill
On 02/06/2018 11:02 PM, Alexandre Oliva wrote: On Feb 6, 2018, Jason Merrill wrote: On 12/11/2017 09:52 PM, Alexandre Oliva wrote: Why do we need to use a non-zero view identifier for a zero view? Why can't we always use 0 instead of the bitmap? We assign view ids

[Bug target/84220] rs6000 builtin __builtin_vec_sld() ICEs on invalid 3rd argument

2018-02-07 Thread bergner at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84220 Peter Bergner changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED URL|

Re: RFA: Sanitize deprecation messages (PR 84195)

2018-02-07 Thread David Malcolm
On Wed, 2018-02-07 at 17:26 +, Nick Clifton wrote: > Hi Martin, Hi David, > > OK - attached is a new patch that: > > * Replaces control characters with their escape equivalents. > * Includes a testcase. > > I was not sure what to do about the inconsistencies between the > behaviour of

[Bug fortran/84270] New: optimization bug with assumed size array argument

2018-02-07 Thread weinert at cmth dot phys.uwm.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84270 Bug ID: 84270 Summary: optimization bug with assumed size array argument Product: gcc Version: 7.3.1 Status: UNCONFIRMED Severity: normal Priority: P3

[PATCH][GCC][ARM] Silence more re-definition warnings, make test case failure case more explicit.

2018-02-07 Thread Tamar Christina
Hi All, The previous testcase would fail on a system where the initial mode is thumb and later switches to an arm mode. This would again cause some warnings to be emitted. This patch visits all builtins defined with builtin_define_with_int_value and undefines them if they could possibly change

Re: [PATCH, rs6000] Update vsx-vector-6-le.c tests for p9 target

2018-02-07 Thread Segher Boessenkool
Hi! On Wed, Feb 07, 2018 at 11:16:12AM -0600, Will Schmidt wrote: > Noted during review of test results on P9. Due to changes and improvements, > our codegen is different for this test on power9. > Modified the existing test to target P8, and added a P9 variant with updated > counts. > diff

Re: gdb 8.x - g++ 7.x compatibility

2018-02-07 Thread Simon Marchi
On 2018-02-07 12:30, Jonathan Wakely wrote: Ah ok, the class name appears mangled in other entities' mangled name. But from what I understand there's no mangled name for the class such that echo | c++filt outputs the class name (e.g. "Foo<10>"). That wouldn't make sense, since there's

Re: [PATCH] C++: avoid most reserved words as misspelling suggestions (PR c++/81610 and PR c++/80567)

2018-02-07 Thread David Malcolm
On Wed, 2018-02-07 at 13:22 -0500, Jason Merrill wrote: > On Wed, Feb 7, 2018 at 1:12 PM, David Malcolm > wrote: > > On Wed, 2018-02-07 at 12:21 -0500, Jason Merrill wrote: > > > On Fri, Jan 26, 2018 at 1:12 PM, David Malcolm > > om> > > > wrote: > > > >

Re: C++ PATCH to fix ICE with vector expr folding (PR c++/83659)

2018-02-07 Thread Jason Merrill
On Wed, Feb 7, 2018 at 12:54 PM, Marek Polacek wrote: > On Wed, Feb 07, 2018 at 12:26:04PM -0500, Jason Merrill wrote: >> On Fri, Jan 26, 2018 at 6:22 PM, Jakub Jelinek wrote: >> > On Fri, Jan 26, 2018 at 02:11:19PM +0100, Richard Biener wrote: >> >> >>

Re: [PATCH] C++: avoid most reserved words as misspelling suggestions (PR c++/81610 and PR c++/80567)

2018-02-07 Thread Jason Merrill
On Wed, Feb 7, 2018 at 1:12 PM, David Malcolm wrote: > On Wed, 2018-02-07 at 12:21 -0500, Jason Merrill wrote: >> On Fri, Jan 26, 2018 at 1:12 PM, David Malcolm >> wrote: >> > On Mon, 2017-12-11 at 17:24 -0500, Jason Merrill wrote: >> > > On Wed, Nov 22,

[Bug preprocessor/81419] GCC wrongly suggests function-like macro as fixit hint for undefined object-like macro

2018-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81419 David Malcolm changed: What|Removed |Added Status|NEW |ASSIGNED CC|

[Bug target/83789] __builtin_altivec_lvx fails for powerpc for altivec-4.c

2018-02-07 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83789 --- Comment #4 from Segher Boessenkool --- Kaushik: is this fixed with r256762?

Re: [PATCH] C++: avoid most reserved words as misspelling suggestions (PR c++/81610 and PR c++/80567)

2018-02-07 Thread David Malcolm
On Wed, 2018-02-07 at 12:21 -0500, Jason Merrill wrote: > On Fri, Jan 26, 2018 at 1:12 PM, David Malcolm > wrote: > > On Mon, 2017-12-11 at 17:24 -0500, Jason Merrill wrote: > > > On Wed, Nov 22, 2017 at 10:36 AM, David Malcolm > > com> > > > wrote: > > >

[Bug c++/80567] [8 Regression] bogus fixit hint for undeclared memset: else

2018-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80567 David Malcolm changed: What|Removed |Added See Also||https://gcc.gnu.org/bugzill

  1   2   3   >