[Bug fortran/97046] Bad interaction between lbound/ubound, allocatable arrays and bind(C) subroutine with dimension(..) parameter

2020-09-14 Thread igor.gayday at mu dot edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97046 --- Comment #5 from Igor Gayday --- I'd like to add that Gilles Gouaillardet is the author of the reproducer in my original post.

Re: PowerPC long double Mangling

2020-09-14 Thread Michael Meissner via Gcc
On Wed, Sep 09, 2020 at 02:42:36PM +0100, Jonathan Wakely wrote: > Sorry for the slow reply to this. > > On Fri, 7 Aug 2020 at 22:14, Michael Meissner wrote: > > > > One issue with doing the transition is what mangling should be used with the > > new long double. > > > > At the moment, the

[Bug c++/97055] New: Copy and move constructors shadowed by templatized constructor

2020-09-14 Thread amir.ahmed.ansari at outlook dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97055 Bug ID: 97055 Summary: Copy and move constructors shadowed by templatized constructor Product: gcc Version: 10.2.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH] rs6000: inefficient 64-bit constant generation for consecutive 1-bits

2020-09-14 Thread Alan Modra via Gcc-patches
On Thu, Sep 10, 2020 at 04:58:03PM -0500, Peter Bergner via Gcc-patches wrote: > +unsigned long > +test0 (void) > +{ > + return 0x0000UL; > +} > + > +unsigned long > +test1 (void) > +{ > + return 0x0000UL; > +} > + > +unsigned long > +test2 (void) > +{ > + return

[Bug target/97054] [r10-3559 Regression] Runtime segfault with attached test code

2020-09-14 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97054 H.J. Lu changed: What|Removed |Added Status|UNCONFIRMED |NEW CC|

[Bug target/97054] New: [r10-3559 Regression] Runtime segfault with attached test code

2020-09-14 Thread skpgkp2 at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97054 Bug ID: 97054 Summary: [r10-3559 Regression] Runtime segfault with attached test code Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal

Re: [PATCH v2] rs6000: Expand vec_insert in expander instead of gimple [PR79251]

2020-09-14 Thread luoxhu via Gcc-patches
On 2020/9/14 17:47, Richard Biener wrote: On Mon, Sep 14, 2020 at 10:05 AM luoxhu wrote: Not sure whether this reflects the issues you discussed above. I constructed below test cases and tested with and without this patch, only if "a+c"(which means store only), the performance is getting

Re: Ping: [PATCH 1/2] Fold plusminus_mult expr with multi-use operands (PR 94234)

2020-09-14 Thread Feng Xue OS via Gcc-patches
>@@ -3426,8 +3426,16 @@ dt_simplify::gen_1 (FILE *f, int indent, bool >gimple, operand *result) > /* Re-fold the toplevel result. It's basically an embedded > gimple_build w/o actually building the stmt. */ > if (!is_predicate) >- fprintf_indent (f,

[Bug tree-optimization/94234] missed ccp folding for (addr + 8 * n) - (addr + 8 * (n - 1))

2020-09-14 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94234 --- Comment #4 from CVS Commits --- The master branch has been updated by Feng Xue : https://gcc.gnu.org/g:f9d2def016410a2095df6b399097b482f82064a5 commit r11-3199-gf9d2def016410a2095df6b399097b482f82064a5 Author: Feng Xue Date: Tue Sep 1

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-09-14 Thread Qing Zhao via Gcc-patches
> On Sep 14, 2020, at 6:09 PM, Segher Boessenkool > wrote: > > On Fri, Sep 11, 2020 at 05:41:47PM -0500, Qing Zhao wrote: >>> On Sep 11, 2020, at 4:51 PM, Segher Boessenkool >>> wrote: >>> It is definitely *not* effective if there are gadgets that set rax to >>> a value the attacker wants

[PATCH] Return mask <-> integer cost for non-AVX512 micro-architecture.

2020-09-14 Thread Hongtao Liu via Gcc-patches
Hi: This patch would avoid spill gprs to mask registers for non-AVX512 micro-architecture and fix regression in PR96744. Bootstrap is ok, regression test for i386/x86-64 backend is ok. No big performance impact on SPEC2017. gcc/ChangeLog: PR taregt/96744 *

[Bug c++/97053] New: an O3 codegen bug

2020-09-14 Thread cuzdav at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97053 Bug ID: 97053 Summary: an O3 codegen bug Product: gcc Version: 10.2.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++ Assignee:

[Bug target/97035] csky-linux-gnuabiv2 won't build with binutils master branch

2020-09-14 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97035 H.J. Lu changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug c++/97051] Evaluating is_constant_evaluated in requires clause fails

2020-09-14 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97051 --- Comment #2 from Marek Polacek --- This compiles when __builtin_is_constant_evaluated is used instead.

[Bug c++/97051] Evaluating is_constant_evaluated in requires clause fails

2020-09-14 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97051 Marek Polacek changed: What|Removed |Added Last reconfirmed||2020-09-15 Keywords|

[Bug c++/97052] Internal compiler error with substitution failure in template parameter list of concept declaration

2020-09-14 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97052 Marek Polacek changed: What|Removed |Added Last reconfirmed||2020-09-15

[Bug c++/97052] New: Internal compiler error with substitution failure in template parameter list of concept declaration

2020-09-14 Thread david at doublewise dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97052 Bug ID: 97052 Summary: Internal compiler error with substitution failure in template parameter list of concept declaration Product: gcc Version: 11.0 Status: UNCONFIRMED

[Bug target/97042] powerpc64 UINT_MAX constant

2020-09-14 Thread amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97042 --- Comment #9 from Alan Modra --- I think that splitter should disappear and rs6000_emit_set_long_const handle all special cases where you might want combinations of two logical instructions before handling the li;rldicl, li;rldicr or any other

Re: [PATCH v2] c, c++: Implement -Wsizeof-array-div [PR91741]

2020-09-14 Thread Marek Polacek via Gcc-patches
On Mon, Sep 14, 2020 at 08:39:36PM +, Joseph Myers wrote: > On Mon, 14 Sep 2020, Marek Polacek via Gcc-patches wrote: > > > so I followed suit. In the C++ FE this was rather easy, because > > finish_parenthesized_expr already set TREE_NO_WARNING. In the C FE > > it was trickier; I've added

[Bug c++/97051] New: Evaluating is_constant_evaluated in requires clause fails

2020-09-14 Thread david at doublewise dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97051 Bug ID: 97051 Summary: Evaluating is_constant_evaluated in requires clause fails Product: gcc Version: 11.0 Status: UNCONFIRMED Severity: normal

[RS6000] rs6000_rtx_costs for PLUS/MINUS constant

2020-09-14 Thread Alan Modra via Gcc-patches
These functions do behave a little differently for SImode, so the mode should be passed. * config/rs6000/rs6000.c (rs6000_rtx_costs): Pass mode to reg_or_add_cint_operand and reg_or_sub_cint_operand. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index

[RS6000] rs6000_rtx_costs for AND

2020-09-14 Thread Alan Modra via Gcc-patches
The existing "case AND" in this function is not sufficient for optabs.c:avoid_expensive_constant usage, where the AND is passed in outer_code. * config/rs6000/rs6000.c (rs6000_rtx_costs): Move costing for AND to CONST_INT case. diff --git a/gcc/config/rs6000/rs6000.c

[RS6000] rs6000_rtx_costs comment

2020-09-14 Thread Alan Modra via Gcc-patches
Prior patches in this series were small bug fixes. This lays out the ground rules for following patches. * config/rs6000/rs6000.c (rs6000_rtx_costs): Expand comment. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index 523d029800a..5b3c0ee0e8c 100644 ---

[RS6000] rs6000_rtx_costs reduce cost for SETs

2020-09-14 Thread Alan Modra via Gcc-patches
Also use rs6000_cost only for speed. * config/rs6000/rs6000.c (rs6000_rtx_costs): Reduce cost for SETs when insn operation cost handled on recursive call. Only use rs6000_cost for speed. Tidy break/return. Tidy AND costing. diff --git a/gcc/config/rs6000/rs6000.c

[RS6000] rs6000_rtx_costs multi-insn constants

2020-09-14 Thread Alan Modra via Gcc-patches
This small patch to rs6000_rtx_const considerably improves code generated for large constants in 64-bit code, teaching gcc that it is better to load a constant from memory than to generate a sequence of up to five dependent instructions. Note that the rs6000 backend does generate large constants

[RS6000] rotate and mask constants

2020-09-14 Thread Alan Modra via Gcc-patches
Implement more two insn constants. PR 94393 * config/rs6000/rs6000.c (rotate_and_mask_constant): New function. (num_insns_constant_multi, rs6000_emit_set_long_const): Use it here. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index

[RS6000] rs6000_rtx_costs cost IOR

2020-09-14 Thread Alan Modra via Gcc-patches
* config/rs6000/rs6000.c (rs6000_rtx_costs): Cost IOR. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index 8c300b82b11..fb5fe7969a3 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/rs6000.c @@ -21177,6 +21177,7 @@ static bool rs6000_rtx_costs (rtx x,

[RS6000] rtx_costs

2020-09-14 Thread Alan Modra via Gcc-patches
This patch series fixes a number of issues in rs6000_rtx_costs, the aim being to provide costing somewhat closer to reality. Probably the most important patch of the series is patch 4, which just adds a comment. Without the analysis that went into that comment, I found myself making what seemed

[RS6000] Count rldimi constant insns

2020-09-14 Thread Alan Modra via Gcc-patches
rldimi is generated by rs6000_emit_set_long_const when the high and low 32 bits of a 64-bit constant are equal. * config/rs6000/rs6000.c (num_insns_constant_gpr): Count rldimi constants correctly. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index

Re: PSA: Default C++ dialect is now C++17

2020-09-14 Thread Marek Polacek via Gcc-patches
On Mon, Sep 14, 2020 at 11:13:18AM -0400, Jason Merrill via Gcc-patches wrote: > On Mon, Jun 29, 2020 at 1:25 PM Martin Liška wrote: > > > > On 6/29/20 4:57 PM, Marek Polacek wrote: > > > On Mon, Jun 29, 2020 at 09:51:57AM +0200, Martin Liška wrote: > > >> On 6/26/20 9:34 PM, Marek Polacek via

[Bug fortran/97046] Bad interaction between lbound/ubound, allocatable arrays and bind(C) subroutine with dimension(..) parameter

2020-09-14 Thread gilles.gouaillardet at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97046 --- Comment #4 from Gilles Gouaillardet --- I crafted the reproducer based on a previous one that has already been merged, and using a libc subroutine was not an issue back then.

[Bug target/97042] powerpc64 UINT_MAX constant

2020-09-14 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97042 --- Comment #8 from Segher Boessenkool --- (In reply to Alan Modra from comment #7) > and of course, li 0x is li -1 which sets all bits. Erm, yes. Duh. So that g:5d3ae76af13 splitter should not fire for numbers that fit in 32 bits but

[Bug fortran/97046] Bad interaction between lbound/ubound, allocatable arrays and bind(C) subroutine with dimension(..) parameter

2020-09-14 Thread sgk at troutmask dot apl.washington.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97046 --- Comment #3 from Steve Kargl --- On Mon, Sep 14, 2020 at 11:56:18PM +, gilles.gouaillardet at gmail dot com wrote: > This is the libc subroutine > > void sync(void); > > The point here is any subroutine (that will not cause a crash) can

[Bug fortran/96983] [11 regression] ICE compiling gfortran.dg/pr96711.f90 starting with r11-3042

2020-09-14 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96983 --- Comment #26 from Segher Boessenkool --- What Joseph says in c#25. Also, the documentation currently says The @code{KIND} value matches the storage size in bytes, [...] which will have to change, too (the standard does not require this, it

[r11-3192 Regression] FAIL: libgomp.c++/udr-3.C execution test on Linux/x86_64 (-m64)

2020-09-14 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c is the first bad commit commit e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c Author: Nathan Sidwell Date: Mon Sep 14 09:42:29 2020 -0700 c++: local externs in templates do not get template head caused FAIL: libgomp.c++/udr-3.C

[r11-3192 Regression] FAIL: libgomp.c++/udr-13.C execution test on Linux/x86_64 (-m64)

2020-09-14 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c is the first bad commit commit e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c Author: Nathan Sidwell Date: Mon Sep 14 09:42:29 2020 -0700 c++: local externs in templates do not get template head caused FAIL: libgomp.c++/udr-13.C

[Bug fortran/97046] Bad interaction between lbound/ubound, allocatable arrays and bind(C) subroutine with dimension(..) parameter

2020-09-14 Thread gilles.gouaillardet at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97046 Gilles Gouaillardet changed: What|Removed |Added CC||gilles.gouaillardet at gmail dot

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-09-14 Thread Segher Boessenkool
On Mon, Sep 14, 2020 at 05:33:33PM +0100, Richard Sandiford wrote: > > However, for the cases on Power as Segher mentioned, there are also some > > scratch registers used for > > Other purpose, not sure whether we can correctly generate zeroing in > > middle-end for Power? > > Segher would be

[Bug target/97042] powerpc64 UINT_MAX constant

2020-09-14 Thread amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97042 --- Comment #7 from Alan Modra --- and of course, li 0x is li -1 which sets all bits.

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-09-14 Thread Segher Boessenkool
Hi! On Fri, Sep 11, 2020 at 05:24:58PM -0500, Qing Zhao wrote: > > So IMO the pass should just search for all the > > returns in a function and insert the zeroing instructions before > > each one. > > I was considering this approach too for some time, however, there is one > major issue with

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-09-14 Thread Segher Boessenkool
On Fri, Sep 11, 2020 at 05:41:47PM -0500, Qing Zhao wrote: > > On Sep 11, 2020, at 4:51 PM, Segher Boessenkool > > wrote: > > It is definitely *not* effective if there are gadgets that set rax to > > a value the attacker wants and then do a syscall. > > You mean the following gadget: > > >

Re: [PATCH] -fprofile-reproducible: fix option value handling

2020-09-14 Thread Sergei Trofimovich via Gcc-patches
On Mon, 14 Sep 2020 09:34:08 +0200 Richard Biener wrote: > On Fri, Sep 11, 2020 at 11:56 PM Sergei Trofimovich via Gcc-patches > wrote: > > > > From: Sergei Trofimovich > > > > Before the change option handling did not accept an argument: > > xgcc: error: unknown profile reproducibility

[Bug fortran/96041] [11 regression] ICE in gfortran.dg/pr93423.f90 after r11-1792

2020-09-14 Thread burnus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96041 Tobias Burnus changed: What|Removed |Added CC||burnus at gcc dot gnu.org --- Comment

[Bug target/97042] powerpc64 UINT_MAX constant

2020-09-14 Thread amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97042 --- Comment #6 from Alan Modra --- That's easy. rs6000_emit_set_long_const doesn't generate that sequence. Incidentally, a patch I had to generate more constants from li;rldicl also fixes this pr.

Re: [Patch] OpenMP/Fortran: Fix (re)mapping of allocatable/pointer arrays [PR96668]

2020-09-14 Thread Tobias Burnus
On 9/14/20 1:03 PM, Jakub Jelinek wrote: + + if (flag_openmp && !(TYPE_QUALS (TREE_TYPE (ptr)) & TYPE_QUAL_RESTRICT)) +always_modifier = true; I think we don't want to depend on flag_openmp here, ... Guess for now an openacc flag with (ctx->region_type & ORT_ACC) != 0 passed to it

Re: c++: local externs in templates do not get template head

2020-09-14 Thread Tobias Burnus
This patch cause run-time fails for g++ -fopenmp libgomp/testsuite/libgomp.c++/udr-13.C The follow-up fix does not help. Namely, in udr-3.C:115: 115 if (t.s != 11 || v.v != 9 || q != 0 || d != 3.0) abort (); (gdb) p t.s $1 = 11 (gdb) p v.v $2 = 4 (gdb) p q $3 = 0 (gdb) p d $4 = 3

[r11-3192 Regression] FAIL: libgomp.c++/udr-13.C execution test on Linux/x86_64 (-m64 -march=cascadelake)

2020-09-14 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c is the first bad commit commit e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c Author: Nathan Sidwell Date: Mon Sep 14 09:42:29 2020 -0700 c++: local externs in templates do not get template head caused FAIL: libgomp.c++/udr-13.C

[r11-3192 Regression] FAIL: libgomp.c++/udr-3.C execution test on Linux/x86_64 (-m64 -march=cascadelake)

2020-09-14 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c is the first bad commit commit e9fdb9a73249f95f3da2d7fde6f268ae12d0d22c Author: Nathan Sidwell Date: Mon Sep 14 09:42:29 2020 -0700 c++: local externs in templates do not get template head caused FAIL: libgomp.c++/udr-3.C

Re: A problem with one instruction multiple latencies and pipelines

2020-09-14 Thread Segher Boessenkool
On Mon, Sep 14, 2020 at 08:35:44PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > >> Although this looks/sounds complicated, the advantage is that everything > >> remains up-to-date. If we instead added a second attribute and only > >> defined it for instructions like *add__,

[PATCH] bb-reorder: Fix for ICEs caused by 69ca5f3a9882

2020-09-14 Thread Segher Boessenkool
After the previous patch we are left with an unreachable BB. This will ICE if either we have -fschedule-fusion, or we do not have peephole2. This fixes it. Okay for trunk? Segher 2020-09-14 Segher Boessenkool PR rtl-optimization/96475 * bb-reorder.c

Re: [PATCH] correct handling of indices into arrays with elements larger than 1 (PR c++/96511)

2020-09-14 Thread Martin Sebor via Gcc-patches
On 9/4/20 11:14 AM, Jason Merrill wrote: On 9/3/20 2:44 PM, Martin Sebor wrote: On 9/1/20 1:22 PM, Jason Merrill wrote: On 8/11/20 12:19 PM, Martin Sebor via Gcc-patches wrote: -Wplacement-new handles array indices and pointer offsets the same: by adjusting them by the size of the element. 

[Bug c++/97049] Cryptic warning "__builtin_memmove pointer overflow between offset ... and size ..." with -m32

2020-09-14 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97049 Martin Sebor changed: What|Removed |Added Known to work||11.0, 9.3.0 Ever confirmed|0

[Bug analyzer/96650] [11 Regression] ICE in on_fact, at analyzer/constraint-manager.cc:1785

2020-09-14 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96650 David Malcolm changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

Re: [PATCH] Fix overflow handling in std::align

2020-09-14 Thread Thomas Rodgers
> On Sep 14, 2020, at 7:30 AM, Ville Voutilainen via Libstdc++ > wrote: > > On Mon, 14 Sep 2020 at 15:49, Glen Fernandes wrote: >> >> On Mon, Sep 14, 2020 at 5:52 AM Ville Voutilainen wrote: >>> On Mon, 14 Sep 2020 at 12:51, Ville Voutilainen >>> wrote: On Mon, 14 Sep 2020 at 09:18,

[Bug libbacktrace/93608] [libbacktrace] Add support for .gnu_debugdata section (aka MiniDebugInfo)

2020-09-14 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93608 Ian Lance Taylor changed: What|Removed |Added Resolution|--- |FIXED Status|UNCONFIRMED

libbacktrace: Add support for MiniDebugInfo

2020-09-14 Thread Ian Lance Taylor via Gcc-patches
This patch to libbacktrace adds support for MiniDebugInfo, as requested in PR 93608. MiniDebugInfo stores compressed symbol tables for an executable, where the executable is otherwise stripped. It is documented at https://fedoraproject.org/wiki/Features/MiniDebugInfo and

[Bug libbacktrace/93608] [libbacktrace] Add support for .gnu_debugdata section (aka MiniDebugInfo)

2020-09-14 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93608 --- Comment #4 from CVS Commits --- The master branch has been updated by Ian Lance Taylor : https://gcc.gnu.org/g:05f40bc4c116ba48843728201bc7290a5e518598 commit r11-3196-g05f40bc4c116ba48843728201bc7290a5e518598 Author: Ian Lance Taylor

Re: [PATCH v2] rs6000: Expand vec_insert in expander instead of gimple [PR79251]

2020-09-14 Thread Segher Boessenkool
On Mon, Sep 14, 2020 at 11:47:52AM +0100, Richard Sandiford wrote: > Would it be worth changing the optab so that the input and output are > separate? Having a single operand would be justified if the operation > was only supposed to touch the selected bytes, but most targets wouldn't > guarantee

Re: [PATCH v2] rs6000: Expand vec_insert in expander instead of gimple [PR79251]

2020-09-14 Thread Segher Boessenkool
On Mon, Sep 14, 2020 at 11:47:56AM +0200, Richard Biener wrote: > this should be > >u[n % 4] = i; > > I guess. Is the % 4 mandated by the vec_insert semantics btw? Yes: VEC_INSERT (ARG1, ARG2, ARG3) Purpose: Returns a copy of vector ARG2 with element ARG3 replaced by the value of

[Bug fortran/96983] [11 regression] ICE compiling gfortran.dg/pr96711.f90 starting with r11-3042

2020-09-14 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96983 --- Comment #25 from joseph at codesourcery dot com --- On Mon, 14 Sep 2020, anlauf at gcc dot gnu.org wrote: > Remember that Fortran needs a correspondence between a storage representation > (in bytes / bits) and the kind type on the language

Re: [PATCH] c, c++: Implement -Wsizeof-array-div [PR91741]

2020-09-14 Thread Joseph Myers
On Mon, 14 Sep 2020, Marek Polacek via Gcc-patches wrote: > so I followed suit. In the C++ FE this was rather easy, because > finish_parenthesized_expr already set TREE_NO_WARNING. In the C FE > it was trickier; I've added a NOP_EXPR to discern between the non-() > and () versions. This sort

[Bug c++/97050] ICE with segfault in lambda overload resolution

2020-09-14 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97050 --- Comment #2 from Marek Polacek --- Reduced: namespace a { template constexpr bool f = __is_same_as(d, e); } struct g {}; struct h; template auto operator+(i, j) { auto k = [](auto l) requires a::f{}; return k; } void m() { struct n {

Re: [PATH 3/3] libstdc++: Add std::advance ostreambuf_iterator overload

2020-09-14 Thread François Dumont via Gcc-patches
On 10/09/20 5:19 pm, Jonathan Wakely wrote: On 09/09/20 22:12 +0200, François Dumont via Libstdc++ wrote: libstdc++: Add std::advance overload for ostreambuf_iterator Implement std::advance overload for ostreambuf_iterator using basic_streambuf pubseekof. libstdc++-v3/ChangeLog:      

Re: [PATCH] Ignore the clobbered stack pointer in asm statment

2020-09-14 Thread H.J. Lu via Gcc-patches
On Mon, Sep 14, 2020 at 10:05 AM Jakub Jelinek wrote: > > On Mon, Sep 14, 2020 at 08:57:18AM -0700, H.J. Lu via Gcc-patches wrote: > > Something like this for GCC 8 and 9. > > Guess my preference would be to do this everywhere and then let's discuss if The same patch also works on master branch.

Re: [PATCH] mixing of labels and code in C2X

2020-09-14 Thread Joseph Myers
On Sun, 13 Sep 2020, Uecker, Martin wrote: > Hi Joseph, > > here is the (unfinished) patch to support > mixing of labels in C2X. I think there should be explicit tests for old standard versions (c11-labels-1.c etc.) that these usages are errors with -pedantic-errors with the old -std option,

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-09-14 Thread Qing Zhao via Gcc-patches
> On Sep 14, 2020, at 2:20 PM, Richard Sandiford > wrote: > > Qing Zhao mailto:qing.z...@oracle.com>> writes: >>> On Sep 14, 2020, at 11:33 AM, Richard Sandiford >>> wrote: >>> >>> Qing Zhao writes: > Like I mentioned earlier though, passes that run after >

Re: [PATCH v2] rs6000: Expand vec_insert in expander instead of gimple [PR79251]

2020-09-14 Thread Segher Boessenkool
On Thu, Sep 10, 2020 at 12:08:44PM +0200, Richard Biener wrote: > On Wed, Sep 9, 2020 at 6:03 PM Segher Boessenkool > wrote: > > There often are problems over function calls (where the compiler cannot > > usually *see* how something is used). > > Yep. The best way would be to use small loads

[Bug c++/97050] ICE with segfault in lambda overload resolution

2020-09-14 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97050 Marek Polacek changed: What|Removed |Added Last reconfirmed||2020-09-14

[Bug c++/97050] New: ICE with segfault in lambda overload resolution

2020-09-14 Thread numerical.simulation at web dot de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97050 Bug ID: 97050 Summary: ICE with segfault in lambda overload resolution Product: gcc Version: og10 (devel/omp/gcc-10) Status: UNCONFIRMED Severity: normal Priority: P3

[Bug fortran/96983] [11 regression] ICE compiling gfortran.dg/pr96711.f90 starting with r11-3042

2020-09-14 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96983 --- Comment #24 from anlauf at gcc dot gnu.org --- I've submitted the patch in comment#19 for review: https://gcc.gnu.org/pipermail/fortran/2020-September/055079.html This would also change the affected testcase to include comment#2, and defer

[PATCH] PR/fortran 96983 - ICE compiling gfortran.dg/pr96711.f90

2020-09-14 Thread Harald Anlauf
Dear all, my fix for PR fortran/96711 adds a testcase that failed on powerpc64-*-* as well as sparc*-sun-solaris2.11. This is a consequence of the, say, mess, on x86, where we have real kinds 4,8,10,16, with kind=10 being long double and kind=16 being represented as float128, while on sparc

Re: A problem with one instruction multiple latencies and pipelines

2020-09-14 Thread Richard Sandiford
Segher Boessenkool writes: >> Although this looks/sounds complicated, the advantage is that everything >> remains up-to-date. If we instead added a second attribute and only >> defined it for instructions like *add__, other instructions >> (including config/arm instructions) would still have

[Bug middle-end/97048] [meta-bug] bogus/missing -Wstringop-overread warnings

2020-09-14 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97048 Bug 97048 depends on bug 81437, which changed state. Bug 81437 Summary: missing -Wstringop-overread reading past the end of a string https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81437 What|Removed |Added

[Bug tree-optimization/81437] missing -Wstringop-overread reading past the end of a string

2020-09-14 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81437 Martin Sebor changed: What|Removed |Added Target Milestone|--- |11.0 Blocks|88443

[Bug middle-end/97048] [meta-bug] bogus/missing -Wstringop-overread warnings

2020-09-14 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97048 Bug 97048 depends on bug 82456, which changed state. Bug 82456 Summary: missing -Wstringop-overread on strcpy reading past the end of an array https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82456 What|Removed

[Bug tree-optimization/82456] missing -Wstringop-overread on strcpy reading past the end of an array

2020-09-14 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82456 Martin Sebor changed: What|Removed |Added Known to fail|10.0|10.2.0 Assignee|unassigned at

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-09-14 Thread Richard Sandiford
Qing Zhao writes: >> On Sep 14, 2020, at 11:33 AM, Richard Sandiford >> wrote: >> >> Qing Zhao writes: Like I mentioned earlier though, passes that run after pass_thread_prologue_and_epilogue can use call-clobbered registers that weren't previously used. For example, on

[COMMITTED 1/2] bpf: use the expected instruction for NOPs

2020-09-14 Thread Jose E. Marchesi via Gcc-patches
The BPF ISA doesn't have a no-operation instruction, but in practice the Linux kernel verifier performs some optimizations that rely on these instructions to be encoded in a particular way. As it turns out, we were using the "wrong" instruction in GCC. This patch makes GCC to generate the

[Bug rtl-optimization/96475] direct threaded interpreter with computed gotos generates suboptimal dispatch loop

2020-09-14 Thread dinuxbg at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96475 --- Comment #23 from Dimitar Dimitrov --- (In reply to Segher Boessenkool from comment #20) > Could you guys please test the attached patch? Thanks in advance! Yes, it fixed all new regressions for pru-elf. Thanks.

Re: [PATCH 1/2] AArch64: Cleanup CPU option processing code

2020-09-14 Thread Wilco Dijkstra
Hi Richard, >On 14/09/2020 15:19, Wilco Dijkstra wrote: >> The --with-cpu/--with-arch configure option processing not only checks valid >> arguments >> but also sets TARGET_CPU_DEFAULT with a CPU and extension bitmask.  This >> isn't used >> however since a --with-cpu is translated into a -mcpu

[Bug fortran/96983] [11 regression] ICE compiling gfortran.dg/pr96711.f90 starting with r11-3042

2020-09-14 Thread anlauf at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96983 --- Comment #23 from anlauf at gcc dot gnu.org --- (In reply to jos...@codesourcery.com from comment #22) > Closely related: the LONG_DOUBLE_TYPE_SIZE target macro which assumes > "size in bits" can uniquely determine the format of long double.

Re: c++: local externs in templates do not get template head

2020-09-14 Thread Marek Polacek via Gcc-patches
On Mon, Sep 14, 2020 at 12:53:18PM -0400, Nathan Sidwell wrote: > On 9/14/20 12:49 PM, Marek Polacek wrote: > > On Mon, Sep 14, 2020 at 12:45:33PM -0400, Nathan Sidwell wrote: > > > Now we consistently mark local externs with DECL_LOCAL_DECL_P, we can > > > teach the template machinery not to give

[pushed] Darwin, X86, testsuite: Fix pr87767 tests for Darwin.

2020-09-14 Thread Iain Sandoe
Hi The tests assume non-PIC for m32 (which means that they fail on default PIC targets, like Darwin), also on Linux with -fpic, FWIW. There is also a missing space before the closing '}' on one of the dg-require- effective-target which means that test fails on machines without avx512f. Tested

Re: PING [Patch][Middle-end]Add -fzero-call-used-regs=[skip|used-gpr|all-gpr|used|all]

2020-09-14 Thread Qing Zhao via Gcc-patches
> On Sep 14, 2020, at 11:33 AM, Richard Sandiford > wrote: > > Qing Zhao writes: >>> Like I mentioned earlier though, passes that run after >>> pass_thread_prologue_and_epilogue can use call-clobbered registers that >>> weren't previously used. For example, on x86_64, the function might

Re: A problem with one instruction multiple latencies and pipelines

2020-09-14 Thread Segher Boessenkool
Hi! On Mon, Sep 14, 2020 at 10:55:35AM +0100, Richard Sandiford wrote: > "Qian, Jianhua" writes: > > - If we cannot resolve it, the existing CPUs' descriptions need > > to be changed. This is not what I expected. > > - If we want to add new attribute to resolve this

[Bug c++/97049] New: Cryptic warning "__builtin_memmove pointer overflow between offset ... and size ..." with -m32

2020-09-14 Thread officesamurai at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97049 Bug ID: 97049 Summary: Cryptic warning "__builtin_memmove pointer overflow between offset ... and size ..." with -m32 Product: gcc Version: 10.2.0 Status: UNCONFIRMED

[PATCH] c, c++: Implement -Wsizeof-array-div [PR91741]

2020-09-14 Thread Marek Polacek via Gcc-patches
This patch implements a new warning, -Wsizeof-array-div. It warns about code like int arr[10]; sizeof (arr) / sizeof (short); where we have a division of two sizeof expressions, where the first argument is an array, and the second sizeof does not equal the size of the array element. See

[Bug c++/91741] Implement new warning -Wsizeof-array-div

2020-09-14 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91741 Marek Polacek changed: What|Removed |Added Keywords||patch --- Comment #3 from Marek Polacek

Re: Dominance information problem

2020-09-14 Thread Gary Oblock via Gcc
Erick, I assume that this needs to be done on all the functions since you mention "cfun". Gary From: Erick Ochoa Sent: Monday, September 14, 2020 12:10 AM To: Gary Oblock ; gcc@gcc.gnu.org Subject: Re: Dominance information problem [EXTERNAL EMAIL NOTICE: This

Re: [Bug target/96968] aarch64 : ICE in vregs or expand pass, lowering __builtin_aarch64_get_{fpcr,fpsr,fpcr64,fpsr64}

2020-09-14 Thread Andrea Corallo
Patch posted at: https://gcc.gnu.org/pipermail/gcc-patches/2020-September/553882.html

[Bug target/96968] aarch64 : ICE in vregs or expand pass, lowering __builtin_aarch64_get_{fpcr,fpsr,fpcr64,fpsr64}

2020-09-14 Thread andrea.corallo at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96968 --- Comment #4 from Andrea Corallo --- Patch posted at: https://gcc.gnu.org/pipermail/gcc-patches/2020-September/553882.html

Re: [PATCH]rs6000: Remove useless insns fed into lvx/stvx [PR97019]

2020-09-14 Thread Segher Boessenkool
Hi! On Mon, Sep 14, 2020 at 05:53:13PM +0800, Kewen.Lin wrote: > This patch is to extend the existing function find_alignment_op to > check all defintions of base_reg are AND operations with mask -16B > to force the alignment. If all are satifised, it passes all AND > operations and instructions

[Bug fortran/97046] Bad interaction between lbound/ubound, allocatable arrays and bind(C) subroutine with dimension(..) parameter

2020-09-14 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97046 kargl at gcc dot gnu.org changed: What|Removed |Added CC||kargl at gcc dot gnu.org ---

[Bug fortran/96983] [11 regression] ICE compiling gfortran.dg/pr96711.f90 starting with r11-3042

2020-09-14 Thread joseph at codesourcery dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96983 --- Comment #22 from joseph at codesourcery dot com --- On Fri, 11 Sep 2020, segher at gcc dot gnu.org wrote: > > #ifndef RS6000_MODES_H > > #define RS6000_MODES_H 1 > > #define FLOAT_PRECISION_IFmode 128 > > #define

Re: [PATCH] Ignore the clobbered stack pointer in asm statment

2020-09-14 Thread Jakub Jelinek via Gcc-patches
On Mon, Sep 14, 2020 at 08:57:18AM -0700, H.J. Lu via Gcc-patches wrote: > Something like this for GCC 8 and 9. Guess my preference would be to do this everywhere and then let's discuss if we change the warning into error there or keep it being deprecated. Though, let's see what others want to

Import license issue

2020-09-14 Thread Andrew Stubbs
Hi All, I need to update include/hsa.h to access some newer APIs. The existing file was created by copying from the user manual, thus side-stepping licensing issues, but the updated user manual omits some important details from the APIs I need (mostly the contents of structs and value of

Re: c++: local externs in templates do not get template head

2020-09-14 Thread Nathan Sidwell
On 9/14/20 12:49 PM, Marek Polacek wrote: On Mon, Sep 14, 2020 at 12:45:33PM -0400, Nathan Sidwell wrote: Now we consistently mark local externs with DECL_LOCAL_DECL_P, we can teach the template machinery not to give them a TEMPLATE_DECL head, and the instantiation machinery treat them as the

Re: c++: local externs in templates do not get template head

2020-09-14 Thread Marek Polacek via Gcc-patches
On Mon, Sep 14, 2020 at 12:45:33PM -0400, Nathan Sidwell wrote: > Now we consistently mark local externs with DECL_LOCAL_DECL_P, we can > teach the template machinery not to give them a TEMPLATE_DECL head, > and the instantiation machinery treat them as the local specialiations > they are.

[Bug analyzer/96653] -Wanalyzer-too-complex on very large switch statement

2020-09-14 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96653 David Malcolm changed: What|Removed |Added Summary|Compile time and memory hog |-Wanalyzer-too-complex on

c++: local externs in templates do not get template head

2020-09-14 Thread Nathan Sidwell
Now we consistently mark local externs with DECL_LOCAL_DECL_P, we can teach the template machinery not to give them a TEMPLATE_DECL head, and the instantiation machinery treat them as the local specialiations they are. (openmp UDRs also fall into this category, and are dealt with similarly.)

  1   2   3   >