Re: [RFC,PATCH] Introduce -msdata=explicit for powerpc

2018-08-07 Thread Alexandre Oliva
On Aug 7, 2018, Segher Boessenkool wrote: > Hi! Hi! > On Tue, Aug 07, 2018 at 02:18:59AM -0300, Alexandre Oliva wrote: >> I saw comments, docs and init code that suggested the possibility of >> using r2/.sdata2 for small data, but I couldn't get code to be generated >> for such access, even

[Bug c/86885] New: gcc erroneously allows constructor/destructor attributes on nested functions

2018-08-07 Thread bugdal at aerifal dot cx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86885 Bug ID: 86885 Summary: gcc erroneously allows constructor/destructor attributes on nested functions Product: gcc Version: unknown Status: UNCONFIRMED Severity:

Re: [PATCH] Make strlen range computations more conservative

2018-08-07 Thread Martin Sebor
On 08/07/2018 11:44 AM, Richard Biener wrote: On August 7, 2018 4:37:00 PM GMT+02:00, Martin Sebor wrote: On 08/07/2018 02:51 AM, Richard Biener wrote: On August 7, 2018 4:24:42 AM GMT+02:00, Martin Sebor wrote: On 07/26/2018 02:55 AM, Richard Biener wrote: On Wed, 25 Jul 2018, Martin

[Bug tree-optimization/86884] New: aggressive loop optimization and effective type

2018-08-07 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86884 Bug ID: 86884 Summary: aggressive loop optimization and effective type Product: gcc Version: 9.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

[Bug c/84890] Overly verbose notes for missing headers

2018-08-07 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84890 Eric Gallager changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #10 from Eric

[Bug c++/86878] G++ should warn about invalid alignments passed to allocation functions

2018-08-07 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86878 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/86878] G++ should warn about invalid alignments passed to allocation functions

2018-08-07 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86878 Martin Sebor changed: What|Removed |Added CC||msebor at gcc dot gnu.org --- Comment #2

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

2018-08-07 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81419 --- Comment #2 from Eric Gallager --- (In reply to Martin Sebor from comment #1) > Confirmed. There are few other similar bugs for these fix-it hints (e.g., > pr80567 or pr80684). I haven't looked at the implementation but from the > symptoms

[Bug tree-optimization/80147] missing maybe-uninitialized warning on variable with no side effects

2018-08-07 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80147 --- Comment #6 from Eric Gallager --- (In reply to Manuel López-Ibáñez from comment #2) > (In reply to Eric Gallager from comment #1) > > ...but if that's the case, then why doesn't the warning go away for i1, too? > > Because "is-used" is

Re: [PATCH, rs6000] Correct descriptions of __builtin_bcdadd* and _builtin_bcdsub* functions

2018-08-07 Thread Segher Boessenkool
Hi Kelvin, On Tue, Aug 07, 2018 at 05:42:48PM -0500, Kelvin Nilsen wrote: > > My "consistency" check was against the implementation. > > On 8/2/18 11:38 AM, Segher Boessenkool wrote: > > On Wed, Aug 01, 2018 at 02:55:22PM -0500, Kelvin Nilsen wrote: > >> Several errors were discovered in the

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-07 Thread Martin Sebor
On 08/07/2018 05:31 AM, Joseph Myers wrote: On Tue, 7 Aug 2018, Martin Sebor wrote: 2) skipping embedded nuls made it possible to create a string with fewer elements than the initializer array, which caused arrays with unspecified bound to be smaller than they would have been otherwise I

Re: [PATCH] convert braced initializers to strings (PR 71625)

2018-08-07 Thread Martin Sebor
On 08/07/2018 02:57 AM, Jason Merrill wrote: On Wed, Aug 1, 2018 at 12:49 AM, Martin Sebor wrote: On 07/31/2018 07:38 AM, Jason Merrill wrote: On Tue, Jul 31, 2018 at 9:51 AM, Martin Sebor wrote: The middle-end contains code to determine the lengths of constant character arrays

Re: [PATCH, rs6000] Early gimple folding of vec_mergeh and vec_mergel for float

2018-08-07 Thread Segher Boessenkool
Hi! On Tue, Aug 07, 2018 at 02:24:58PM -0500, Will Schmidt wrote: >This adds support for gimple folding of vec_mergeh and vec_mergel > for float and double types. Support for the integral types is already > in-tree. > + /* The permute_type will match the lhs for integral types. For

[Bug libstdc++/84654] libstdc++ tries to use __float128 when compiling with -mno-float128

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84654 Jonathan Wakely changed: What|Removed |Added Target Milestone|8.3 |7.4 --- Comment #8 from Jonathan

Re: [PATCH v2 01/10] Initial TI PRU GCC port

2018-08-07 Thread Sandra Loosemore
On 07/28/2018 07:44 AM, Dimitar Dimitrov wrote: ChangeLog: 2018-07-27 Dimitar Dimitrov * configure: Regenerate. * configure.ac: Add PRU target. gcc/ChangeLog: 2018-07-27 Dimitar Dimitrov * common/config/pru/pru-common.c: New file. * config.gcc: Add PRU

[Bug libstdc++/68519] condition_variable::wait_for does not work properly with float duration

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68519 Jonathan Wakely changed: What|Removed |Added Target Milestone|8.0 |7.4 --- Comment #11 from Jonathan

[Bug libstdc++/86292] Missing exception safety when constructing vector from input iterator pair

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86292 Jonathan Wakely changed: What|Removed |Added Target Milestone|8.2 |7.4 --- Comment #7 from Jonathan

[Bug libstdc++/86861] 18_support/new_aligned.cc FAILs

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86861 Jonathan Wakely changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug libstdc++/84654] libstdc++ tries to use __float128 when compiling with -mno-float128

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84654 --- Comment #7 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:49 2018 New Revision: 263382 URL: https://gcc.gnu.org/viewcvs?rev=263382=gcc=rev Log: PR libstdc++/84654 Disable __float128 specializations for -mno-float128 Backport

[Bug libstdc++/86138] [7/8 Regression] C++17: getline(istream, string) crashes on Cygwin because incompatible C++14 function is called

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86138 --- Comment #28 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:55 2018 New Revision: 263383 URL: https://gcc.gnu.org/viewcvs?rev=263383=gcc=rev Log: Declare some explicit instantiations for strings in Debug Mode The empty reps

[Bug libstdc++/86734] [DR 2188] reverse_iterator::operator-> does not support overloaded operator

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86734 Jonathan Wakely changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug libstdc++/85672] [9 Regression] error: redefinition of 'constexpr long double std::abs(long double)'

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85672 --- Comment #16 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:49 2018 New Revision: 263382 URL: https://gcc.gnu.org/viewcvs?rev=263382=gcc=rev Log: PR libstdc++/84654 Disable __float128 specializations for -mno-float128 Backport

[Bug libstdc++/86861] 18_support/new_aligned.cc FAILs

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86861 --- Comment #8 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:19 2018 New Revision: 263376 URL: https://gcc.gnu.org/viewcvs?rev=263376=gcc=rev Log: PR libstdc++/86861 Meet precondition for Solaris memalign Solaris memalign

[Bug libstdc++/80893] std::vector creation dereferences null pointer

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80893 --- Comment #6 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:45 2018 New Revision: 263381 URL: https://gcc.gnu.org/viewcvs?rev=263381=gcc=rev Log: PR libstdc++/80893 Fix null dereference in vector Backport from mainline

[Bug libstdc++/86292] Missing exception safety when constructing vector from input iterator pair

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86292 --- Comment #6 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:33 2018 New Revision: 263379 URL: https://gcc.gnu.org/viewcvs?rev=263379=gcc=rev Log: PR libstdc++/86292 fix exception safety of std::vector constructor Backport from

[Bug libstdc++/68519] condition_variable::wait_for does not work properly with float duration

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68519 --- Comment #10 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:40 2018 New Revision: 263380 URL: https://gcc.gnu.org/viewcvs?rev=263380=gcc=rev Log: PR libstdc++/68519 use native duration to avoid rounding errors Backport from

[Bug libstdc++/60555] std::system_category().default_error_condition() doesn't map system errno values to std::generic_category()

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60555 --- Comment #14 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:24 2018 New Revision: 263377 URL: https://gcc.gnu.org/viewcvs?rev=263377=gcc=rev Log: PR libstdc++/60555 std::system_category() should recognise POSIX errno values

[Bug libstdc++/86734] [DR 2188] reverse_iterator::operator-> does not support overloaded operator

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86734 --- Comment #4 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 22:50:11 2018 New Revision: 263374 URL: https://gcc.gnu.org/viewcvs?rev=263374=gcc=rev Log: PR libstdc++/86734 use addressof in reverse_iterator::operator-> Backport from

[Bug debug/86593] [8/9 Regression] internal compiler error: in based_loc_descr, at dwarf2out.c:14272

2018-08-07 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86593 --- Comment #5 from H.J. Lu --- This test: --- struct Foo { int bar(int a, int b, int c, int i1, int i2, int i3, int d); }; int Foo::bar(int a, int b, int c, int i1, int i2, int i3, int d) { return 0; } --- also triggers

[Bug c++/86862] Segfault using extern template on class deriving from streambuf

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86862 Jonathan Wakely changed: What|Removed |Added Keywords||ice-on-valid-code

Re: [PATCH, rs6000] Correct descriptions of __builtin_bcdadd* and _builtin_bcdsub* functions

2018-08-07 Thread Kelvin Nilsen
My "consistency" check was against the implementation. On 8/2/18 11:38 AM, Segher Boessenkool wrote: > Hi Kelvin, > > On Wed, Aug 01, 2018 at 02:55:22PM -0500, Kelvin Nilsen wrote: >> Several errors were discovered in the descriptions of the __builtin_bcdadd, >> __builtin_bcdadd_lt,

Re: PR libstdc++/68222 Hide safe iterator operators

2018-08-07 Thread Jonathan Wakely
On 07/08/18 14:47 +0100, Jonathan Wakely wrote: On 02/08/18 22:16 +0200, François Dumont wrote: Hi     Here is a patch to avoid definition of invalid operators on the Debug mode safe iterator type depending on its category.     Even if it is limited to the Debug mode code I would like to

Re: PR libstdc++/68222 Hide safe iterator operators

2018-08-07 Thread Jonathan Wakely
On 02/08/18 22:16 +0200, François Dumont wrote: +#if __cplusplus >= 201103L + /** @brief Copy assignment. */ + _Safe_tagged_iterator& + operator=(const _Safe_tagged_iterator&) = default; + + /** @brief Move assignment. */ + _Safe_tagged_iterator& +

[Bug c++/86883] Unexpected error: expansion pattern '' contains no argument packs

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86883 --- Comment #1 from Jonathan Wakely --- See also PR 68342 and PR 85569

[Bug c++/86883] Unexpected error: expansion pattern '' contains no argument packs

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86883 --- Comment #2 from Jonathan Wakely --- And PR 86859

[Bug c++/86883] New: Unexpected error: expansion pattern '' contains no argument packs

2018-08-07 Thread v.reshetnikov at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86883 Bug ID: 86883 Summary: Unexpected error: expansion pattern '' contains no argument packs Product: gcc Version: 8.2.0 Status: UNCONFIRMED Severity: normal

[PATCH][OpenACC] Update deviceptr handling during gimplification

2018-08-07 Thread Cesar Philippidis
I had previously posted this patch as part of a monster deviceptr patch here . This patch breaks out the generic gimplifier changes. Essentially, with this patch, the gimplifier will now transfer deviceptr data clauses using

[Bug c++/86879] G++ should warn about redundant tests for null pointers returned from functions with __attribute__((returns_nonnull))

2018-08-07 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86879 Martin Sebor changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[PATCH][OpenACC] Don't error on implicitly private induction variables in gfortran

2018-08-07 Thread Cesar Philippidis
At present, the fortran FE reports an error if the user adds an explicit private clause to an induction variable used by an acc loop. This patch teaches the fortran acc block resolver how to cope with "duplicate" private clauses, so that it doesn't error anymore. Is this patch OK for trunk? I

[Bug debug/86593] [8/9 Regression] internal compiler error: in based_loc_descr, at dwarf2out.c:14272

2018-08-07 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86593 --- Comment #4 from H.J. Lu --- This patch seems to work diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c index b67481dd2db..6ecdc4562d0 100644 --- a/gcc/dwarf2out.c +++ b/gcc/dwarf2out.c @@ -14304,7 +14304,8 @@ based_loc_descr (rtx reg,

[PATCH][OpenACC] Add support for firstprivate Fortran allocatable scalars

2018-08-07 Thread Cesar Philippidis
This patch updates the way that lower_omp_target uses firstprivate pointers in OpenACC offloaded regions. On host side, when preparing firstprivate data mapping for pointer type objects, not to be confused with GOMP_MAP_FIRSTPRIVATE_POINTER, the compiler passes passes the address of the value

Re: [PATCH, rs6000] testcases for GIMPLE folding of vec_splat builtin.

2018-08-07 Thread Segher Boessenkool
Hi! On Tue, Aug 07, 2018 at 02:25:06PM -0500, Will Schmidt wrote: > Some testcases to exercise the vec_splat() built-in. > In building and updating these tests I consciously violated the > 80 char per line rule, as I was doing experimentation with the > assorted values and making lots of local

[Bug libstdc++/84535] std::thread constructor missing constraint on first argument

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84535 Jonathan Wakely changed: What|Removed |Added Target Milestone|9.0 |8.3 --- Comment #5 from Jonathan

[PATCH][OpenACC] update gfortran's tile clause error handling

2018-08-07 Thread Cesar Philippidis
This patch updates how the OpenACC tile clause is handled in the Fortran FE to match it's behavior in C/C++. Specifically, the tile clause now errors on negative integer arguments, instead of emitting a warning. Is this OK for trunk? Thanks, Cesar >From af39a6d65cfb46397fa62c88521189002fb3d705

[Bug libstdc++/60555] std::system_category().default_error_condition() doesn't map system errno values to std::generic_category()

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60555 Jonathan Wakely changed: What|Removed |Added Target Milestone|9.0 |8.3

[Bug libstdc++/86874] [8/9 Regression] std::swap on std::variant fails to compile

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86874 Jonathan Wakely changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[PATCH][OpenACC] cleanup trans-stmt.h

2018-08-07 Thread Cesar Philippidis
This patch removes a stale reference to trans-openacc.c in gcc/fortran/trans-statement.h. I'll apply it to trunk as obvious shortly. Cesar >From a08fe168c3f3ca4d446915ad26027786cda58394 Mon Sep 17 00:00:00 2001 From: Cesar Philippidis Date: Tue, 14 Mar 2017 22:33:00 + Subject: [PATCH]

[Bug libstdc++/84535] std::thread constructor missing constraint on first argument

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84535 --- Comment #4 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 21:38:59 2018 New Revision: 263370 URL: https://gcc.gnu.org/viewcvs?rev=263370=gcc=rev Log: PR libstdc++/84535 constrain std::thread constructor The standard requires that

[Bug libstdc++/86861] 18_support/new_aligned.cc FAILs

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86861 --- Comment #7 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 21:38:45 2018 New Revision: 263367 URL: https://gcc.gnu.org/viewcvs?rev=263367=gcc=rev Log: PR libstdc++/86861 Meet precondition for Solaris memalign Solaris memalign

[Bug libstdc++/60555] std::system_category().default_error_condition() doesn't map system errno values to std::generic_category()

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60555 --- Comment #13 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 21:38:55 2018 New Revision: 263369 URL: https://gcc.gnu.org/viewcvs?rev=263369=gcc=rev Log: PR libstdc++/60555 std::system_category() should recognise POSIX errno values

[Bug libstdc++/86874] [8/9 Regression] std::swap on std::variant fails to compile

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86874 --- Comment #3 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 21:38:50 2018 New Revision: 263368 URL: https://gcc.gnu.org/viewcvs?rev=263368=gcc=rev Log: PR libstdc++/86874 fix std::variant::swap regression Backport from mainline

Re: [PATCH] Line map table allocation

2018-08-07 Thread Nathan Sidwell
On 08/07/2018 03:43 PM, David Malcolm wrote: OK for trunk, with the nits noted above fixed. Committing this, thanks nathan -- Nathan Sidwell 2018-08-07 Nathan Sidwell * line-map.c: (linemap_init): Set default allocator here. (new_linemap): Rather than here. Refactor allocation logic.

Re: [PATCH] Add sinh(tanh(x)) and cosh(tanh(x)) rules

2018-08-07 Thread Giuliano Augusto Faulin Belinassi
That is a good question because I didn't know that such targets exists. Any suggestion? On Tue, Aug 7, 2018 at 5:29 PM, Paul Koning wrote: > > >> On Aug 7, 2018, at 4:00 PM, Giuliano Augusto Faulin Belinassi >> wrote: >> >> Related with bug 86829, but for hyperbolic trigonometric functions.

Re: [PATCH] Add sinh(tanh(x)) and cosh(tanh(x)) rules

2018-08-07 Thread Paul Koning
> On Aug 7, 2018, at 4:00 PM, Giuliano Augusto Faulin Belinassi > wrote: > > Related with bug 86829, but for hyperbolic trigonometric functions. > This patch adds substitution rules to both sinh(tanh(x)) -> x / sqrt(1 > - x*x) and cosh(tanh(x)) -> 1 / sqrt(1 - x*x). Notice that the both >

Re: [RFC][PATCH][mid-end] Optimize immediate choice in comparisons.

2018-08-07 Thread Richard Sandiford
Hi Vlad, Thanks for the patch. Vlad Lazar writes: > Hi. > > This patch optimises the choice of immediates in integer comparisons. Integer > comparisons allow for different choices (e.g. a > b is equivalent to a >= b+1) > and there are cases where an incremented/decremented immediate can be

[PATCH] Add sinh(tanh(x)) and cosh(tanh(x)) rules

2018-08-07 Thread Giuliano Augusto Faulin Belinassi
Related with bug 86829, but for hyperbolic trigonometric functions. This patch adds substitution rules to both sinh(tanh(x)) -> x / sqrt(1 - x*x) and cosh(tanh(x)) -> 1 / sqrt(1 - x*x). Notice that the both formulas has division by 0, but it causes no harm because 1/(+0) -> +infinity, thus the

Re: [PATCH, testsuite]: Fix g++.dg/torture/pr86763.C link failure for glibc < 2.17

2018-08-07 Thread Rainer Orth
Hi Andreas, > How about replacing clock_gettime with something else? It's not needed > for the particular test. that would certainly be useful: Solaris 10 also needs librt or clock_gettime, while Solaris 11 has folded it into libc. Rainer --

Re: [PATCH] Line map table allocation

2018-08-07 Thread David Malcolm
On Mon, 2018-08-06 at 12:58 -0400, Nathan Sidwell wrote: > Here's a line-map patch to make the new_linemap logic simpler. On > the > modules branch I need to allocate blocks of linemaps, and found the > current allocation scheme a little confusing to adjust. This'll > make > that subsequent

[PATCH, rs6000] testcases for GIMPLE folding of vec_splat builtin.

2018-08-07 Thread Will Schmidt
Hi, Some testcases to exercise the vec_splat() built-in. Tested successfully on assorted powerpc Linux systems, in conjunction with the vec-splat() folding patch, which is being posted separately. In building and updating these tests I consciously violated the 80 char per line rule, as

[PATCH, rs6000] Early gimple folding of vec_mergeh and vec_mergel for float

2018-08-07 Thread Will Schmidt
Hi, This adds support for gimple folding of vec_mergeh and vec_mergel for float and double types. Support for the integral types is already in-tree. This change includes updates to the fold_mergehl_helper function to handle creating the (integral) permute vector when the vector type non

[PATCH, RFC, rs6000] enable GIMPLE folding of vec_splat

2018-08-07 Thread Will Schmidt
Hi Enable GIMPLE folding of the vec_splat() intrinsic. For review.. feedback is expected. :-) I came up with the following after spending some time poking around at the tree_vec_extract() and vector_element() functions as seen in tree-vect-generic.c looking for insights. Some of this

[Bug inline-asm/59155] ICE: in reg_overlap_mentioned_p, at rtlanal.c:1473

2018-08-07 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59155 Arseny Solokha changed: What|Removed |Added CC||asolokha at gmx dot com --- Comment #9

[Bug rtl-optimization/86882] New: [9 Regression] ICE in reg_overlap_mentioned_p, at rtlanal.c:1873

2018-08-07 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86882 Bug ID: 86882 Summary: [9 Regression] ICE in reg_overlap_mentioned_p, at rtlanal.c:1873 Product: gcc Version: 9.0 Status: UNCONFIRMED Keywords:

[Bug libstdc++/86874] [8/9 Regression] std::swap on std::variant fails to compile

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86874 --- Comment #2 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 19:13:26 2018 New Revision: 263365 URL: https://gcc.gnu.org/viewcvs?rev=263365=gcc=rev Log: PR libstdc++/86874 fix std::variant::swap regression PR libstdc++/86874

Re: [PATCH] fix std::variant::swap for trivially-move-assignable types

2018-08-07 Thread Jonathan Wakely
On 07/08/18 17:57 +0300, Ville Voutilainen wrote: On 7 August 2018 at 17:29, Jonathan Wakely wrote: On 07/08/18 15:24 +0100, Jonathan Wakely wrote: This patch fixes the bug, but is it correct? IIUC the _M_destructive_move effects don't depend on whether move assignment is trivial, so should

[Bug c++/86881] New: tree check fail with flag Wshadow-compatible-local

2018-08-07 Thread dcb314 at hotmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86881 Bug ID: 86881 Summary: tree check fail with flag Wshadow-compatible-local Product: gcc Version: 8.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component:

Oracle E-Business Users Contact List

2018-08-07 Thread Meghan Hudson
Hi, Hope you having a great day! I just wanted to be aware if you looking to acquire Oracle E-Business Users Contact List for your marketing efforts? We also have users of: Microsoft Dynamics 365, Ab Initio, SAS Enterprise, Deltek Vision, Zucchetti and many more... Kindly review and

Re: [PATCH] Make strlen range computations more conservative

2018-08-07 Thread Richard Biener
On August 7, 2018 6:31:36 PM GMT+02:00, Martin Sebor wrote: >On 08/07/2018 09:33 AM, Bernd Edlinger wrote: >> On 08/07/18 17:02, Martin Sebor wrote: >>> On 08/06/2018 11:45 PM, Richard Biener wrote: On August 7, 2018 5:38:59 AM GMT+02:00, Martin Sebor > wrote: > On 08/06/2018 11:40 AM,

Re: [PATCH] Make strlen range computations more conservative

2018-08-07 Thread Richard Biener
On August 7, 2018 4:37:00 PM GMT+02:00, Martin Sebor wrote: >On 08/07/2018 02:51 AM, Richard Biener wrote: >> On August 7, 2018 4:24:42 AM GMT+02:00, Martin Sebor > wrote: >>> On 07/26/2018 02:55 AM, Richard Biener wrote: On Wed, 25 Jul 2018, Martin Sebor wrote: >> BUT - for the

[Bug libstdc++/60441] Incorrect textual representation for std::mersenne_twister_engine

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60441 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug libstdc++/86880] Incorrect mersenne_twister_engine equality comparison between rotated states

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86880 Jonathan Wakely changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug target/86838] [9 Regression] ICE in gen_aarch64_frecpe, at ./insn-opinit.h:571

2018-08-07 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86838 rsandifo at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED

Re: [Patch][Aarch64] Implement Aarch64 SIMD ABI and aarch64_vector_pcs attribute

2018-08-07 Thread Segher Boessenkool
Hi! Some very trivial comments... On Mon, Aug 06, 2018 at 03:13:52PM -0700, Steve Ellcey wrote: > (aarch64_components_for_bb): Check for simd function. > (aarch64_epilogue_uses): New function. > (aarch64_process_components): Ditto. > (aarch64_expand_prologue): Ditto. >

Re: [PING 3][PATCH] [v4][aarch64] Avoid tag collisions for loads falkor

2018-08-07 Thread Siddhesh Poyarekar
On 08/07/2018 10:30 PM, James Greenhalgh wrote: To help set expectations here. I'm currently only able to dedicate a couple of hours of time to review each week. Tamar's Stack Clash has been taking a big chunk of that time recently as we push it to a final state for trunk. This pass is... large

libgo patch committed: Uncomment trace.Stop call in testing package

2018-08-07 Thread Ian Lance Taylor
This patch by Than McIntosh uncomments the call to trace.Stop in the testing package. It was commented out when the runtime/trace package did not build with gccgo, but now it does and has for some time. The fixes the go test -test.trace option. Bootstrapped and ran Go testsuite on

[Bug debug/86593] [8/9 Regression] internal compiler error: in based_loc_descr, at dwarf2out.c:14272

2018-08-07 Thread manisandro at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86593 Sandro Mani changed: What|Removed |Added CC||hjl.tools at gmail dot com --- Comment #3

[Bug target/86838] [9 Regression] ICE in gen_aarch64_frecpe, at ./insn-opinit.h:571

2018-08-07 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86838 --- Comment #3 from rsandifo at gcc dot gnu.org --- Author: rsandifo Date: Tue Aug 7 17:22:19 2018 New Revision: 263362 URL: https://gcc.gnu.org/viewcvs?rev=263362=gcc=rev Log: [AArch64] Fix -mlow-precision-div (PR 86838) The "@" handling

[Bug tree-optimization/86858] [9 Regression] GCC ICE at -O3 in as_a, at is-a.h:197

2018-08-07 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86858 rsandifo at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug c++/86871] [8/9 Regression] ICE: gimple check: expected gimple_assign(error_mark), have gimple_call(trunc_mod_expr) in gimple_assign_lhs, at gimple.h:2462

2018-08-07 Thread rsandifo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86871 rsandifo at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug debug/86593] [8/9 Regression] internal compiler error: in based_loc_descr, at dwarf2out.c:14272

2018-08-07 Thread manisandro at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86593 --- Comment #2 from Sandro Mani --- Started with commit cd557ff63f388ad27c376d0a225e74d3594a6f9d Author: hjl Date: Thu Aug 10 15:29:05 2017 + i386: Don't use frame pointer without stack access When there is no stack access,

Re: [PATCH] [Ada] Make middle-end string literals NUL terminated

2018-08-07 Thread Bernd Edlinger
On 08/07/18 18:40, Olivier Hainque wrote: > Hi Bernd, > > (Thanks for your interest in the Ada case as well :) > >> On 7 Aug 2018, at 15:07, Bernd Edlinger wrote: > >> When I try this example: >> >> $ cat array9.adb >> -- { dg-do run } >> >> procedure Array9 is >> >>V1 : String(1..10) :=

Re: [PATCH][Middle-end] disable strcmp/strncmp inlining with O2 below and Os

2018-08-07 Thread Qing Zhao
Hi, Christophe, I have attached a patch in PR86519, could you please download it and test it, and let me know the result. thanks. Qing > On Jul 30, 2018, at 8:45 AM, Christophe Lyon > wrote: > > On Wed, 25 Jul 2018 at 19:08, Qing Zhao > wrote: >> >> Hi, >> >>

Re: [PING 3][PATCH] [v4][aarch64] Avoid tag collisions for loads falkor

2018-08-07 Thread James Greenhalgh
On Tue, Aug 07, 2018 at 03:01:28AM -0500, Siddhesh Poyarekar wrote: > Hello, > > Ping! To help set expectations here. I'm currently only able to dedicate a couple of hours of time to review each week. Tamar's Stack Clash has been taking a big chunk of that time recently as we push it to a final

[Bug testsuite/86519] [9 Regression] New test case gcc.dg/strcmpopt_6.c fails with its introduction in r262636

2018-08-07 Thread qinzhao at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86519 --- Comment #16 from qinzhao at gcc dot gnu.org --- please test the proposed patch on your platform, let me know the result.

[Bug testsuite/86519] [9 Regression] New test case gcc.dg/strcmpopt_6.c fails with its introduction in r262636

2018-08-07 Thread qinzhao at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86519 --- Comment #15 from qinzhao at gcc dot gnu.org --- Created attachment 44516 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44516=edit proposed patch

Re: [PATCH] [aarch64][v2] Fix falkor pipeline description for dup

2018-08-07 Thread James Greenhalgh
On Thu, Aug 02, 2018 at 09:02:05PM -0500, Siddhesh Poyarekar wrote: > On 08/03/2018 12:02 AM, James Greenhalgh wrote: > > On Thu, Aug 02, 2018 at 11:58:37AM -0500, Siddhesh Poyarekar wrote: > >> There was a typo in the pipeline description where DUP was assigned to > >> the vector pipes for quad

[Bug c++/59480] Missing error diagnostic: friend declaration specifying a default argument must be a definition

2018-08-07 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59480 Paolo Carlini changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

[Bug c++/59480] Missing error diagnostic: friend declaration specifying a default argument must be a definition

2018-08-07 Thread paolo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59480 --- Comment #21 from paolo at gcc dot gnu.org --- Author: paolo Date: Tue Aug 7 16:40:18 2018 New Revision: 263361 URL: https://gcc.gnu.org/viewcvs?rev=263361=gcc=rev Log: /cp 2018-08-07 Paolo Carlini PR c++/59480, DR 136 *

Re: [PATCH] [Ada] Make middle-end string literals NUL terminated

2018-08-07 Thread Olivier Hainque
Hi Bernd, (Thanks for your interest in the Ada case as well :) > On 7 Aug 2018, at 15:07, Bernd Edlinger wrote: > When I try this example: > > $ cat array9.adb > -- { dg-do run } > > procedure Array9 is > > V1 : String(1..10) := "1234567890"; > V2 : String(1..-1) := ""; > > procedure

Re: dejagnu version update?

2018-08-07 Thread Segher Boessenkool
On Mon, Aug 06, 2018 at 08:25:49AM -0700, Mike Stump wrote: > Since g++ already requires 1.5.3, it make no sense to bump to anything older > that 1.5.3, so let's bump to 1.5.3. Those packaging systems and OSes that > wanted to update by now, have had their chance to update. Those that punt >

Re: dejagnu version update?

2018-08-07 Thread Segher Boessenkool
On Mon, Aug 06, 2018 at 08:25:49AM -0700, Mike Stump wrote: > Since g++ already requires 1.5.3, it make no sense to bump to anything older > that 1.5.3, so let's bump to 1.5.3. Those packaging systems and OSes that > wanted to update by now, have had their chance to update. Those that punt >

Re: [PATCH] Make strlen range computations more conservative

2018-08-07 Thread Martin Sebor
On 08/07/2018 09:33 AM, Bernd Edlinger wrote: On 08/07/18 17:02, Martin Sebor wrote: On 08/06/2018 11:45 PM, Richard Biener wrote: On August 7, 2018 5:38:59 AM GMT+02:00, Martin Sebor wrote: On 08/06/2018 11:40 AM, Jeff Law wrote: On 08/06/2018 11:15 AM, Martin Sebor wrote: These examples

Re: [PATCH][GCC][AArch64] Validate and set default parameters for stack-clash. [Patch (3/3)]

2018-08-07 Thread James Greenhalgh
On Tue, Aug 07, 2018 at 05:09:30AM -0500, Tamar Christina wrote: > Hi All, > > This is an updated patch which applies the same style changes as requested in > patch 5/6. No full stop on an error message IIRC. Otherwise, OK. Thanks, James > gcc/ > 2018-08-07 Tamar Christina > > *

Re: [PATCH][GCC][AArch64] Ensure that outgoing argument size is at least 8 bytes when alloca and stack-clash. [Patch (3/6)]

2018-08-07 Thread James Greenhalgh
On Tue, Aug 07, 2018 at 05:09:34AM -0500, Tamar Christina wrote: > Hi All, > > This is a re-spin to address review comments. No code change aside from a > variable rename. > > Ok for trunk? OK. Thanks, James > gcc/ > 2018-08-07 Tamar Christina > > PR target/86486 > *

[Bug c++/86878] G++ should warn about invalid alignments passed to allocation functions

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86878 --- Comment #1 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 16:10:29 2018 New Revision: 263360 URL: https://gcc.gnu.org/viewcvs?rev=263360=gcc=rev Log: PR libstdc++/86861 Meet precondition for Solaris memalign Solaris memalign

[Bug libstdc++/86861] 18_support/new_aligned.cc FAILs

2018-08-07 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86861 --- Comment #6 from Jonathan Wakely --- Author: redi Date: Tue Aug 7 16:10:29 2018 New Revision: 263360 URL: https://gcc.gnu.org/viewcvs?rev=263360=gcc=rev Log: PR libstdc++/86861 Meet precondition for Solaris memalign Solaris memalign

[PATCH] PR libstdc++/86861 Meet precondition for Solaris memalign

2018-08-07 Thread Jonathan Wakely
Solaris memalign requires alignment to be at least sizeof(int), so increase it as needed. Also move the check for valid alignments from the fallback implementation of aligned_alloc into operator new, as it's required for all of aligned_alloc, memalign, posix_memalign and __aligned_malloc. This

Re: [Patch][Aarch64] Implement Aarch64 SIMD ABI and aarch64_vector_pcs attribute

2018-08-07 Thread Steve Ellcey
I have a question about my own patch.  In doing testing I realized that if I use the aarch64_vector_pcs attribute on a platform without SIMD (i.e. -march=armv8.1-a+nosimd) I get an ICE.  That is obviously not what we want but I was wondering what the right behaviour is. We certainly don't want to

Re: [PATCH] Make strlen range computations more conservative

2018-08-07 Thread Jeff Law
On 08/06/2018 05:59 PM, Martin Sebor wrote: >>> But as I said, far more essential than the optimization is >>> the ability to detect these invalid access (both reads and >>> writes), such as in: >> >> The essential thing is to not introduce latent wrong code issues >> because you exploit C

Re: [PATCH][GCC][AARCH64] Use STLUR for atomic_store

2018-08-07 Thread Matthew Malcomson
Hello everyone, This is an updated patch where I've made what was a predicate into a pure C function based on some feedback outside the mailing list. Ok for trunk? Matthew gcc/ 2018-08-07  Matthew Malcomson      * config/aarch64/aarch64-protos.h    

  1   2   >