*PING* [PATCH] PR fortran/82314 - ICE in gfc_conv_expr_descriptor, at fortran/trans-array.c:6972

2021-09-12 Thread Harald Anlauf via Gcc-patches
Early *PING*. > Gesendet: Dienstag, 07. September 2021 um 23:44 Uhr > Von: "Harald Anlauf" > An: "fortran" , "gcc-patches" > Betreff: [PATCH] PR fortran/82314 - ICE in gfc_conv_expr_descriptor, at > fortran/trans-array.c:6972 > > When adding the initializer for an array, we need to make sure

Re: [PATCH] rs6000: Disable optimizing multiple xxsetaccz instructions into one xxsetaccz

2021-09-12 Thread Bill Schmidt via Gcc-patches
Hi Peter, This patch looks fine to me.  The approach to avoiding incorrect optimization is reasonable.  Maintainers? Thanks for the patch! Bill On 8/27/21 2:58 PM, Peter Bergner via Gcc-patches wrote: Fwprop will happily optimize two xxsetaccz instructions into one xxsetaccz by propagating

Re: [PATCH, rs6000] optimization for vec_reve builtin [PR100868]

2021-09-12 Thread Bill Schmidt via Gcc-patches
Hi Haochen, On 9/8/21 1:42 AM, HAO CHEN GUI wrote: Hi,   The patch optimized for vec_reve builtin on rs6000. For V2DI and V2DF, it is implemented by xxswapd on all targets. For V16QI, V8HI, V4SI and V4SF, it is implemented by quadword byte reverse plus halfword/word byte reverse when

[committed] d: Don't include terminating null pointer in string expression conversion (PR102185)

2021-09-12 Thread Iain Buclaw via Gcc-patches
Hi, This patch fixes an issue with the routine that converts STRING_CST to a StringExp for the dmd front-end to use during the semantic pass. The null terminator gets re-added by the ExprVisitor when lowering StringExp back into a STRING_CST during the code generator pass. Bootstrapped and

Re: [PATCH v2] ipa-inline: Add target info into fn summary [PR102059]

2021-09-12 Thread Bill Schmidt via Gcc-patches
Hi Kewen, I'll leave the continued review of the back-end parts of this to Segher, but I do have one long-term comment.  The rs6000_builtin_info[code].mask field that you're relying on is going away as part of the built-in function rewrite.  There will be a "bifattrs" field that replaces this

[PATCH] PR fortran/85130 - Handling of substring range

2021-09-12 Thread Harald Anlauf via Gcc-patches
Dear all, in find_substring_ref we erroneously handled given substring start and end indices as unsigned integers. However, gives indices could be negative, which is legal as long as end < start, leading to a string of length zero. The current behavior could lead to a wrong length as well as an

[PATCH] Fix PR lto/49664: liblto_plugin.so exports too many symbols

2021-09-12 Thread apinski--- via Gcc-patches
From: Andrew Pinski So right now liblto_plugin.so exports many libiberty symbols and simple_object file symbols but really it just needs to export onload. This fixes the problem by using "-export-symbols-regex onload" on the libtool link line. lto-plugin/ChangeLog: * Makefile.am:

[PATCH] Fix multi-statement define for alpha-dec-vms

2021-09-12 Thread Jan-Benedict Glaw
Hi! While mass-building a cross-gcc, I noticed that for alpha-dec-vms/alpha64-dec-vms, recent GCC versions correctly throw a warning due to a multi-statement define that gets rippen in an if/else case: [all 2021-09-12 15:51:55] /usr/lib/gcc-snapshot/bin/g++ -fno-PIE -c -g -O2 -DIN_GCC

Re: [PATCH] rs6000: Disable optimizing multiple xxsetaccz instructions into one xxsetaccz

2021-09-12 Thread Segher Boessenkool
Hi! On Fri, Aug 27, 2021 at 02:58:05PM -0500, Peter Bergner wrote: > Fwprop will happily optimize two xxsetaccz instructions into one xxsetaccz > by propagating the results of the first to the uses of the second. > We really don't want that to happen given the late priming/depriming of >

[PATCH] c++: shortcut bad convs during overload resolution, part 2 [PR101904]

2021-09-12 Thread Patrick Palka via Gcc-patches
The r12-3346 patch makes us avoid computing excess argument conversions during overload resolution, but only when it turns out there's a strictly viable candidate in the overload set. If there is no such candidate then we still need to compute more conversions than strictly necessary because

Re: [PATCH] c++: parameter pack inside constexpr if [PR101764]

2021-09-12 Thread Patrick Palka via Gcc-patches
On Thu, 2 Sep 2021, Jason Merrill wrote: > On 8/30/21 10:05 PM, Patrick Palka wrote: > > Here when partially substituting into the pack expansion, substitution > > into the constexpr if yields a still-dependent tree, so tsubst_expr > > returns an IF_STMT with an unsubstituted IF_COND and with > >

[PATCH] Remove UNSPEC_{COPYSIGN,XORSIGN}.

2021-09-12 Thread liuhongt via Gcc-patches
Hi: UNSPEC_COPYSIGN/XORSIGN are only used by related post_reload splitters which have been removed by r12-3417 and r12-3435. Bootstrapped and regtest on x86_64-linux-gnu{-m32,}. Pushed to trunk. gcc/ChangeLog: * config/i386/i386.md: (UNSPEC_COPYSIGN): Remove.

Re: [PATCH] c++: parameter pack inside constexpr if [PR101764]

2021-09-12 Thread Jason Merrill via Gcc-patches
On 9/12/21 7:48 PM, Patrick Palka wrote: On Thu, 2 Sep 2021, Jason Merrill wrote: On 8/30/21 10:05 PM, Patrick Palka wrote: Here when partially substituting into the pack expansion, substitution into the constexpr if yields a still-dependent tree, so tsubst_expr returns an IF_STMT with an

Re: [PATCH] tree-optimization/102155 - fix LIM fill_always_executed_in CFG walk

2021-09-12 Thread Xionghu Luo via Gcc-patches
On 2021/9/10 21:54, Xionghu Luo via Gcc-patches wrote: On 2021/9/9 18:55, Richard Biener wrote: diff --git a/gcc/tree-ssa-loop-im.c b/gcc/tree-ssa-loop-im.c index 5d6845478e7..4b187c2cdaf 100644 --- a/gcc/tree-ssa-loop-im.c +++ b/gcc/tree-ssa-loop-im.c @@ -3074,15 +3074,13 @@

Re: [PATCH] Remove dbx.h, do not set PREFERRED_DEBUGGING_TYPE from dbxcoff.h, lynx.h

2021-09-12 Thread Jan-Benedict Glaw
Hi Richard, On Fri, 2021-09-10 08:02:00 +0200, Richard Biener via Gcc-patches wrote: > > On 9/9/2021 7:19 AM, Richard Biener via Gcc-patches wrote: > > > The patch also removes the PREFERRED_DEBUGGING_TYPE define from > > > lynx.h which always follows elfos.h already defaulting to DWARF, > > >

Re: [PATCH, rs6000] optimization for vec_reve builtin [PR100868]

2021-09-12 Thread Segher Boessenkool
Hi! On Sun, Sep 12, 2021 at 10:50:17AM -0500, Bill Schmidt wrote: > On 9/8/21 1:42 AM, HAO CHEN GUI wrote: > >+;; Vector reverse elements for V2DI V2DF > >+(define_expand "altivec_vreve2" > >+  [(set (match_operand:VEC_64 0 "register_operand" "=v") > >+   (unspec:VEC_64 [(match_operand:VEC_64