Re: [PATCH, rs6000] Fix PR87496: ICE in aggregate_value_p at gcc/function.c:2046

2018-11-29 Thread Peter Bergner
On 11/29/18 1:31 PM, Peter Bergner wrote: > On 11/29/18 11:26 AM, Segher Boessenkool wrote: >> On Wed, Nov 28, 2018 at 03:27:19PM -0600, Peter Bergner wrote: >>> PR87496 shows a bug where we ICE if we attempt to use -mabi=ieeelongdouble >>> and -mno-popcntd. The IEEE128 support requires full ISA 2

Re: [PING #4][PATCH] avoid warning on constant strncpy until next statement is reachable (PR 87028)

2018-11-29 Thread Martin Sebor
On 11/29/18 4:07 PM, Jeff Law wrote: On 11/29/18 1:34 PM, Martin Sebor wrote: On 11/16/2018 02:07 AM, Richard Biener wrote: On Fri, Nov 16, 2018 at 4:12 AM Martin Sebor wrote: Ping: https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01818.html Please let me know if there is something I need to c

Re: [PATCH 1/2] asm qualifiers (PR55681)

2018-11-29 Thread Segher Boessenkool
On Thu, Nov 29, 2018 at 11:14:45PM +, Joseph Myers wrote: > On Thu, 29 Nov 2018, Segher Boessenkool wrote: > > > > What's the basis for allowing duplicates for C but not for C++? > > > > It is the status quo. It would make sense to allow duplicates for C++ as > > well, sure. If that is pref

[PATCH, libstdc++] Uniform container erasure for c++20 returning number of erasures.

2018-11-29 Thread Ed Smith-Rowland
This version of erase_if/erase returning number erased actually build and tests cleanly on x86_64-linux. Is this what you has in mind for this?  I basically return the number of erased items for all the things Ed

Re: [PATCH 1/2] asm qualifiers (PR55681)

2018-11-29 Thread Joseph Myers
On Thu, 29 Nov 2018, Segher Boessenkool wrote: > So "asm const restrict" is allowed, but "asm const const restrict" isn't. No, asm const restrict isn't allowed. volatile is allowed; const and restrict are allowed with warnings because that replicates what the old bison parser allowed; but at m

Re: [PATCH 1/2] asm qualifiers (PR55681)

2018-11-29 Thread Segher Boessenkool
On Fri, Nov 30, 2018 at 12:11:30AM +, Joseph Myers wrote: > On Thu, 29 Nov 2018, Segher Boessenkool wrote: > > > So "asm const restrict" is allowed, but "asm const const restrict" isn't. > > No, asm const restrict isn't allowed. volatile is allowed; const and > restrict are allowed with war

Re: [PING #4][PATCH] avoid warning on constant strncpy until next statement is reachable (PR 87028)

2018-11-29 Thread Jeff Law
On 11/29/18 4:43 PM, Martin Sebor wrote: >> The fallout on existing tests is minimal.  What's more concerning is >> that it doesn't actually pass the new test from Martin's original >> submission.  We get bogus warnings. >> >> At least part of the problem is weakness in maybe_diag_stxncpy_trunc. >>

[PATCH, libstdc++, C++20] Implement P0457R2 String Prefix and Suffix Checking.

2018-11-29 Thread Ed Smith-Rowland
Greetings, This patch implements starts_with and ends_with for basic_string and basic_string_view for C++20. This builds and tests cleanly on x86_64-linux. Ed 2018-11-30 Edward Smith-Rowland <3dw...@verizon.net> Implement P0457R2 String Prefix and Suffix Checking. * incl

Re: [PATCH v3] Make function clone name numbering independent.

2018-11-29 Thread Richard Biener
On Wed, Nov 28, 2018 at 10:09 PM Michael Ploujnikov wrote: > > Continuing from https://gcc.gnu.org/ml/gcc-patches/2018-09/msg00228.html > > It took longer than expected, but I've finally rebased this on top of > the new clone_function_name* API and included the requested > optimizations. > > There

Re: [PATCH] asm non-code template parts (alternative to asm inline)

2018-11-29 Thread Richard Biener
On Thu, Nov 29, 2018 at 5:44 PM Alexander Monakov wrote: > > > On Mon, 15 Oct 2018, Richard Biener wrote: > > > I think it's sound but also note that I think it is logically independent > > > of > > > asm inline (). While it may work for the inlining issue for some kernel > > > examples to asm i

Re: [PATCH, libfortran] PR 88137 Initialize backtrace state once

2018-11-29 Thread Janne Blomqvist
PING! (for the GCC-7 branch, I'll commit it after the 7.4 release) On Thu, Nov 22, 2018 at 11:17 AM Janne Blomqvist wrote: > From backtrace.h for backtrace_create_state: > >Calling this function allocates resources that can not be freed. >There is no backtrace_free_state function. The

Re: [RS6000] num_insns_constant ICE

2018-11-29 Thread Alan Modra
On Thu, Nov 29, 2018 at 01:52:34PM -0600, Segher Boessenkool wrote: > On Sun, Nov 25, 2018 at 10:49:12PM +1030, Alan Modra wrote: > > + while (nregs-- > 0) > > +{ > > + int ins = num_insns_constant_gpr (value); > > You probably should mask "value" here so that it is only one GPR. > Alter

Re: [PING #4][PATCH] avoid warning on constant strncpy until next statement is reachable (PR 87028)

2018-11-29 Thread Richard Biener
On Thu, Nov 29, 2018 at 9:34 PM Martin Sebor wrote: > > On 11/16/2018 02:07 AM, Richard Biener wrote: > > On Fri, Nov 16, 2018 at 4:12 AM Martin Sebor wrote: > >> > >> Ping: https://gcc.gnu.org/ml/gcc-patches/2018-08/msg01818.html > >> > >> Please let me know if there is something I need to chang

Re: [RS6000] PowerPC64 soft-float

2018-11-29 Thread Alan Modra
On Thu, Nov 29, 2018 at 12:15:06PM -0600, Segher Boessenkool wrote: > On Sun, Nov 25, 2018 at 10:50:27PM +1030, Alan Modra wrote: > > This patch aims to prevent long sequences loading soft-float > > constants. For 32-bit, it makes sense to load values inline to a gpr > > with lis, addi, but not so

<    1   2