Re: More aggressive threading causing loop-interchange-9.c regression

2021-09-10 Thread Aldy Hernandez via Gcc-patches
I think things are clear enough to propose a patch. Thanks for everyone's input. I have added some comments and tweaked Michael's patch to include the final BB where we're threading to, in the check. Without this last check, we fail in tree-ssa/cunroll-15.c because the latch becomes pollute

[PATCH] Always default to DWARF2 debugging for RX, even with -mas100-syntax

2021-09-10 Thread Richard Biener via Gcc-patches
The RX port defaults to STABS when -mas100-syntax is used because the AS100 assembler does not support some of the pseudo-ops used by DWARF2 debug emission. Since STABS is going to be deprecated that has to change. The following simply always uses DWARF2, likely leaving -mas100-syntax broken when

[PATCH] Default AVR to DWARF2 debug

2021-09-10 Thread Richard Biener via Gcc-patches
This switches the AVR port to generate DWARF2 debugging info by default since the support for STABS is going to be deprecated for GCC 12. OK for trunk? Thanks, Richard. 2021-09-10 Richard Biener * config/avr/elf.h (PREFERRED_DEBUGGING_TYPE): Remove override, pick up DWARF2_DE

[PATCH] Remove DARWIN_PREFER_DWARF and dead code

2021-09-10 Thread Richard Biener via Gcc-patches
This removes the always defined DARWIN_PREFER_DWARF and the code guarded by it being not defined, removing the possibility to default some i386 darwin configurations to STABS when it would not be defined. OK for trunk? Thanks, Richard. 2021-09-10 Richard Biener * config/darwin.h (DAR

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Richard Biener via Gcc-patches
On Thu, 9 Sep 2021, Kees Cook wrote: > On Thu, Sep 09, 2021 at 10:49:11PM +, Qing Zhao wrote: > > Hi, FYI > > > > I just committed the following patch to gcc upstream: > > > > > > https://gcc.gnu.org/pipermail/gcc-cvs/2021-September/353195.html > > Hurray! Thank you so much for working on

[PATCH][www] Add note that STABS is deprecated

2021-09-10 Thread Richard Biener via Gcc-patches
This amends the caveats section of the GCC 12 release notes according to the STABS deprecation. I'll wait with pushing to the point it reflects the state on trunk rather than projecting what will be the state for GCC 12. --- htdocs/gcc-12/changes.html | 6 ++ 1 file changed, 6 insertions(+)

Re: [PATCH][www] Add note that STABS is deprecated

2021-09-10 Thread Jakub Jelinek via Gcc-patches
On Fri, Sep 10, 2021 at 09:28:16AM +0200, Richard Biener via Gcc-patches wrote: > This amends the caveats section of the GCC 12 release notes according > to the STABS deprecation. > > I'll wait with pushing to the point it reflects the state on trunk > rather than projecting what will be the state

Re: [PATCH][www] Add note that STABS is deprecated

2021-09-10 Thread Richard Biener via Gcc-patches
On Fri, 10 Sep 2021, Jakub Jelinek wrote: > On Fri, Sep 10, 2021 at 09:28:16AM +0200, Richard Biener via Gcc-patches > wrote: > > This amends the caveats section of the GCC 12 release notes according > > to the STABS deprecation. > > > > I'll wait with pushing to the point it reflects the state

[PING] Don't maintain a warning spec for 'UNKNOWN_LOCATION'/'BUILTINS_LOCATION' [PR101574] (was: [PATCH 1/13] v2 [PATCH 1/13] Add support for per-location warning groups (PR 74765))

2021-09-10 Thread Thomas Schwinge
Hi! Ping. On 2021-09-03T21:16:46+0200, Thomas Schwinge wrote: > Martin, thanks for your review. Now need someone to formally approve the > third patch. Again attached for easy reference. Grüße Thomas > On 2021-09-01T18:14:46-0600, Martin Sebor wrote: >> On 9/1/21 1:35 PM, Thomas Schwinge

[PING] Re: [Committed] [PATCH 2/4] (v4) On-demand locations within string-literals

2021-09-10 Thread Thomas Schwinge
Hi! Ping. My patches again attached, for easy reference. Grüße Thomas On 2021-09-03T18:33:37+0200, I wrote: > Hi! > > On 2021-09-02T21:09:54+0200, I wrote: >> On 2021-09-02T15:59:14+0200, I wrote: >>> On 2016-08-05T14:16:58-0400, David Malcolm wrote: Committed to trunk as r239175; I'm

[PING] Re: Fix 'hash_table::expand' to destruct stale Value objects

2021-09-10 Thread Thomas Schwinge
Hi! On 2021-09-01T19:31:19-0600, Martin Sebor via Gcc-patches wrote: > On 8/30/21 4:46 AM, Thomas Schwinge wrote: >> Ping -- we still need to plug the memory leak; see patch attached, and/or >> long discussion here: > > Thanks for answering my questions. I have no concerns with going > forward

[PATCH] analyzer: Define INCLUDE_UNIQUE_PTR

2021-09-10 Thread Maxim Blinov
Un-break the build for AArch64 Darwin. Build currently fails with an error very similar to pr82091: ``` In file included from ../../../gcc-master-wip-apple-si/gcc/analyzer/engine.cc:69: In file included from /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacO

Re: Host and offload targets have no common meaning of address spaces

2021-09-10 Thread Thomas Schwinge
Hi! Ping. Patch again attached for easy reference. Plus, incrementally, the two "should we" questions cited below? Grüße Thomas On 2021-08-24T12:23:07+0200, I wrote: > Hi! > > On 2021-08-19T22:13:56+0200, I wrote: >> On 2021-08-16T10:21:04+0200, Jakub Jelinek wrote: >>> On Mon, Aug 16, 20

[PATCH] middle-end/102269 - avoid auto-init of empty types

2021-09-10 Thread Richard Biener via Gcc-patches
This avoids initializing empty types for which we'll eventually leave a .DEFERRED_INIT call without a LHS. Bootstrap & regtest running on x86_64-unknown-linux-gnu. 2021-09-10 Richard Biener PR middle-end/102269 * gimplify.c (is_var_need_auto_init): Empty types do not need

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Christophe LYON via Gcc-patches
On 10/09/2021 00:49, Qing Zhao via Gcc-patches wrote: Hi, FYI I just committed the following patch to gcc upstream: https://gcc.gnu.org/pipermail/gcc-cvs/2021-September/353195.html Hi, Several of the new tests fail on arm and aarch64 with -mabi=ilp32. On arm: gcc:gcc.dg/dg.exp=c-c++-com

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Martin Liška
On 9/10/21 10:47, Christophe LYON via Gcc-patches wrote: Can you check? It's the following bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102269 Martin

[PING] Re: [Patch][GCC][middle-end] - Generate FRINTZ for (double)(int) under -ffast-math on aarch64

2021-09-10 Thread Jirui Wu via Gcc-patches
Hi, Ping: https://gcc.gnu.org/pipermail/gcc-patches/2021-August/577846.html Ok for master? If OK, can it be committed for me, I have no commit rights. Jirui Wu -Original Message- From: Jirui Wu Sent: Friday, September 3, 2021 12:39 PM To: 'Richard Biener' Cc: Richard Biener ; Andrew Pi

[PATCH Take 2] More NEGATE_EXPR folding in match.pd

2021-09-10 Thread Roger Sayle
Hi Richard, Thanks for suggestion, which cleanly solves the problem I was encountering. This revised patch adds a Boolean simplify argument to tree-ssa-sccvn.c's vn_nary_build_or_lookup_1 to control whether to simplification should be performed before value numbering, updating the callers, but the

[PATCH][RFC] Come up with casm state

2021-09-10 Thread Martin Liška
Hi. We're considering with Richi some changes related to how we emit early debug info. That would probably include changes where the debug info would be streamed to a separate file (different from normal .o output file). That said, we would need switching in between output assembly files. The

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Thomas Schwinge
Hi! On 2021-09-10T10:47:00+0200, Christophe LYON via Gcc-patches wrote: > On 10/09/2021 00:49, Qing Zhao via Gcc-patches wrote: >> I just committed the following patch to gcc upstream: >> >> >> https://gcc.gnu.org/pipermail/gcc-cvs/2021-September/353195.html > Several of the new tests fail on a

[PATCH] middle-end/102273 - avoid ICE with auto-init and nested functions

2021-09-10 Thread Richard Biener via Gcc-patches
This refactors expansion to consider non-decl LHS. I suspect the is_val argument is not needed. Bootstrapped and regtest running on x86_64-unknown-linux-gnu. Richard. 2021-09-10 Richard Biener PR middle-end/102273 * internal-fn.c (expand_DEFERRED_INIT): Always expand non-SSA

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread Richard Biener via Gcc-patches
On Fri, Sep 10, 2021 at 1:50 AM Segher Boessenkool wrote: > > On Thu, Sep 09, 2021 at 08:16:16AM +0200, Richard Biener wrote: > > > I think we should (longer term) get rid of the overloaded meanings and > > > uses of subregs. One fairly simple thing is to make a new rtx code > > > "bit_cast" (or

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread Richard Biener via Gcc-patches
On Fri, Sep 10, 2021 at 5:03 AM Hongtao Liu wrote: > > On Fri, Sep 10, 2021 at 7:49 AM Segher Boessenkool > wrote: > > > > On Thu, Sep 09, 2021 at 08:16:16AM +0200, Richard Biener wrote: > > > > I think we should (longer term) get rid of the overloaded meanings and > > > > uses of subregs. One f

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 6:54 PM Richard Biener wrote: > > On Fri, Sep 10, 2021 at 5:03 AM Hongtao Liu wrote: > > > > On Fri, Sep 10, 2021 at 7:49 AM Segher Boessenkool > > wrote: > > > > > > On Thu, Sep 09, 2021 at 08:16:16AM +0200, Richard Biener wrote: > > > > > I think we should (longer term)

Re: Remove tilegx port

2021-09-10 Thread Richard Biener via Gcc-patches
On Fri, Apr 27, 2018 at 9:32 PM Jeff Law wrote: > > On 04/27/2018 11:42 AM, Richard Biener wrote: > > On April 27, 2018 7:26:19 PM GMT+02:00, Jeff Law wrote: > >> On 04/27/2018 09:36 AM, Joseph Myers wrote: > >>> Since tile support has been removed from the Linux kernel for 4.17, > >>> this patch

[PATCH] Always default to DWARF2_DEBUG if not specified, warn about deprecated STABS

2021-09-10 Thread Richard Biener via Gcc-patches
This makes defaults.h choose DWARF2_DEBUG if PREFERRED_DEBUGGING_TYPE is not specified by the target and NO_DEBUG if DWARF is not supported. It also makes us warn when STABS is enabled and removes the corresponding diagnostic from the Ada frontend. The warnings are pruned from the testsuite outpu

Re: [PATCH, rs6000] Optimization for vec_xl_sext

2021-09-10 Thread Bill Schmidt via Gcc-patches
On 9/10/21 12:45 AM, HAO CHEN GUI wrote: Bill,   Thanks so much for your advice.   I refined the patch and passed the bootstrap and regression test. Just one thing, the test case becomes unsupported on P9 if I set "{ dg-require-effective-target power10_ok }". I just want the test case to

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 7:25 PM Hongtao Liu wrote: > > On Fri, Sep 10, 2021 at 6:54 PM Richard Biener > wrote: > > > > On Fri, Sep 10, 2021 at 5:03 AM Hongtao Liu wrote: > > > > > > On Fri, Sep 10, 2021 at 7:49 AM Segher Boessenkool > > > wrote: > > > > > > > > On Thu, Sep 09, 2021 at 08:16:16A

[PATCH 0/2] Revert r12-3277 since it caused regressions on many other targets.

2021-09-10 Thread liuhongt via Gcc-patches
Hi: Details discussed in https://gcc.gnu.org/pipermail/gcc-patches/2021-September/579170.html. Bootstrapped and regtested on x86_64-linux-gnu{-m32,}. Ok for trunk? liuhongt (2): Revert "Get rid of all float-int special cases in validate_subreg." validate_subreg before call gen_lowpar

[PATCH 1/2] Revert "Get rid of all float-int special cases in validate_subreg."

2021-09-10 Thread liuhongt via Gcc-patches
This reverts commit d2874d905647a1d146dafa60199d440e837adc4d. PR target/102254 PR target/102154 PR target/102211 --- gcc/emit-rtl.c | 40 1 file changed, 40 insertions(+) diff --git a/gcc/emit-rtl.c b/gcc/emit-rtl.c index 77ea8948ee8..ff3b4449b37 100644 -

[PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread liuhongt via Gcc-patches
gcc/ChangeLog: * expmed.c (extract_bit_field_using_extv): validate_subreg before call gen_lowpart. --- gcc/expmed.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/gcc/expmed.c b/gcc/expmed.c index 3143f38e057..10d62d857a8 100644 --- a/gcc/expmed.c +++ b/g

Re: [PATCH 1/2] Revert "Get rid of all float-int special cases in validate_subreg."

2021-09-10 Thread Richard Biener via Gcc-patches
On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: > > This reverts commit d2874d905647a1d146dafa60199d440e837adc4d. OK. Richard. > PR target/102254 > PR target/102154 > PR target/102211 > --- > gcc/emit-rtl.c | 40 > 1 file changed, 40 insertions(+) > > d

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Richard Biener via Gcc-patches
On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: > > gcc/ChangeLog: > > * expmed.c (extract_bit_field_using_extv): validate_subreg > before call gen_lowpart. > --- > gcc/expmed.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/gcc/expmed.c b/gcc/expmed

[PATCH RFC] c++: implement C++17 hardware interference size

2021-09-10 Thread Jason Merrill via Gcc-patches
OK, time to finish this up. The main change relative to the last patch I sent to the list is dropping the -finterference-tune flag and making that behavior the default. Any more comments? The last missing piece of the C++17 standard library is the hardware intereference size constants. Mu

Re: More aggressive threading causing loop-interchange-9.c regression

2021-09-10 Thread Michael Matz via Gcc-patches
Hi, On Fri, 10 Sep 2021, Aldy Hernandez via Gcc-patches wrote: > } > + > + /* Threading through a non-empty latch would cause code to be added "through an *empty* latch". The test in code is correct, though. And for the before/after loops flag you added: we have a cfun->curr_properties

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 9:16 PM Richard Biener via Gcc-patches wrote: > > On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: > > > > gcc/ChangeLog: > > > > * expmed.c (extract_bit_field_using_extv): validate_subreg > > before call gen_lowpart. > > --- > > gcc/expmed.c | 6 +- > >

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Segher Boessenkool
On Fri, Sep 10, 2021 at 03:15:56PM +0200, Richard Biener wrote: > On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: > >if (REG_P (target) > > - && TRULY_NOOP_TRUNCATION_MODES_P (GET_MODE (target), ext_mode)) > > ^^^ > > I wonder if herein lies the problem in that the HFmode "trunca

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Richard Biener via Gcc-patches
On September 10, 2021 3:27:09 PM GMT+02:00, Hongtao Liu wrote: >On Fri, Sep 10, 2021 at 9:16 PM Richard Biener via Gcc-patches > wrote: >> >> On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: >> > >> > gcc/ChangeLog: >> > >> > * expmed.c (extract_bit_field_using_extv): validate_subreg >> >

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 9:27 PM Hongtao Liu wrote: > > On Fri, Sep 10, 2021 at 9:16 PM Richard Biener via Gcc-patches > wrote: > > > > On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: > > > > > > gcc/ChangeLog: > > > > > > * expmed.c (extract_bit_field_using_extv): validate_subreg > > >

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 9:32 PM Richard Biener wrote: > > On September 10, 2021 3:27:09 PM GMT+02:00, Hongtao Liu > wrote: > >On Fri, Sep 10, 2021 at 9:16 PM Richard Biener via Gcc-patches > > wrote: > >> > >> On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: > >> > > >> > gcc/ChangeLog: > >> > >

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Jose E. Marchesi via Gcc-patches
Hi Richard. > On Thu, 9 Sep 2021, Kees Cook wrote: > >> On Thu, Sep 09, 2021 at 10:49:11PM +, Qing Zhao wrote: >> > Hi, FYI >> > >> > I just committed the following patch to gcc upstream: >> > >> > >> > https://gcc.gnu.org/pipermail/gcc-cvs/2021-September/353195.html >> >> Hurray! Thank

Re: [PATCH] Default Alpha/VMS to DWARF2 debugging only

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 12:52 AM, Richard Biener via Gcc-patches wrote: This changes the default debug format for Alpha/VMS to DWARF2 only, skipping emission of VMS debug info which is going do be deprecated for GCC 12 alongside the support for STABS. It looks like other flavors of VMS never used VMS_DE

Re: [PATCH] Default AVR to DWARF2 debug

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 1:13 AM, Richard Biener via Gcc-patches wrote: This switches the AVR port to generate DWARF2 debugging info by default since the support for STABS is going to be deprecated for GCC 12. OK for trunk? Thanks, Richard. 2021-09-10 Richard Biener * config/avr/elf.h (PREF

Re: [PATCH] Always default to DWARF2 debugging for RX, even with -mas100-syntax

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 1:05 AM, Richard Biener wrote: The RX port defaults to STABS when -mas100-syntax is used because the AS100 assembler does not support some of the pseudo-ops used by DWARF2 debug emission. Since STABS is going to be deprecated that has to change. The following simply always uses

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 9:32 PM Richard Biener wrote: > > On September 10, 2021 3:27:09 PM GMT+02:00, Hongtao Liu > wrote: > >On Fri, Sep 10, 2021 at 9:16 PM Richard Biener via Gcc-patches > > wrote: > >> > >> On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: > >> > > >> > gcc/ChangeLog: > >> > >

Re: Remove tilegx port

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 5:50 AM, Richard Biener via Libc-alpha wrote: On Fri, Apr 27, 2018 at 9:32 PM Jeff Law wrote: On 04/27/2018 11:42 AM, Richard Biener wrote: On April 27, 2018 7:26:19 PM GMT+02:00, Jeff Law wrote: On 04/27/2018 09:36 AM, Joseph Myers wrote: Since tile support has been removed

Re: More aggressive threading causing loop-interchange-9.c regression

2021-09-10 Thread Aldy Hernandez via Gcc-patches
On 9/10/21 3:16 PM, Michael Matz wrote: Hi, On Fri, 10 Sep 2021, Aldy Hernandez via Gcc-patches wrote: } + + /* Threading through a non-empty latch would cause code to be added "through an *empty* latch". The test in code is correct, though. Whoops. And for the before/after lo

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

2021-09-10 Thread Xionghu Luo via Gcc-patches
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 @@ fill_always_executed_in_1 (class loop *loop, sbitmap contains_call)

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Richard Biener via Gcc-patches
On September 10, 2021 3:30:10 PM GMT+02:00, Segher Boessenkool wrote: >On Fri, Sep 10, 2021 at 03:15:56PM +0200, Richard Biener wrote: >> On Fri, Sep 10, 2021 at 2:58 PM liuhongt wrote: >> >if (REG_P (target) >> > - && TRULY_NOOP_TRUNCATION_MODES_P (GET_MODE (target), ext_mode))

[PATCH] libsanitizer: Add AM_CCASFLAGS to Makefile.am

2021-09-10 Thread H.J. Lu via Gcc-patches
Add AM_CCASFLAGS to Makefile.am to compile assembly codes with $CET_FLAGS. * asan/Makefile.am (AM_CCASFLAGS): New. Set to $(EXTRA_ASFLAGS). * hwasan/Makefile.am (AM_CCASFLAGS): Likewise. * interception/Makefile.am (AM_CCASFLAGS): Likewise. * lsan/Makefile.am (AM_CC

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 3:08 AM, Martin Liška wrote: On 9/10/21 10:47, Christophe LYON via Gcc-patches wrote: Can you check? It's the following bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=102269 And it's already fixed too ;-)  All my targets with a kernel build component were failing because of

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread David Edelsohn via Gcc-patches
On Thu, Sep 9, 2021 at 11:03 PM Hongtao Liu wrote: > > On Fri, Sep 10, 2021 at 7:49 AM Segher Boessenkool > wrote: > > > > On Thu, Sep 09, 2021 at 08:16:16AM +0200, Richard Biener wrote: > > > > I think we should (longer term) get rid of the overloaded meanings and > > > > uses of subregs. One f

[committed] libstdc++: Use "test.invalid." for invalid hostname

2021-09-10 Thread Jonathan Wakely via Gcc-patches
This avoids test.invalid.some.domain being successfully resolved. libstdc++-v3/ChangeLog: * testsuite/experimental/net/internet/resolver/ops/lookup.cc: Fix invalid hostname to only match the .invalid TLD. Tested x86_64-linux. Committed to trunk. commit 7f8af6dc82a0dac0d97fdd4d1f

Re: [PATCH 0/3] bpf: add -mcpu and related feature options

2021-09-10 Thread Jose E. Marchesi via Gcc-patches
Hi David. > New instructions have been added over time to the eBPF ISA, but > previously there has been no good method to select which version to > target in GCC. > > This patch adds the following options to the BPF backend: > > -mcpu={v1, v2, v3} > Select which version of the eBPF ISA to

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 10:08 PM David Edelsohn wrote: > > On Thu, Sep 9, 2021 at 11:03 PM Hongtao Liu wrote: > > > > On Fri, Sep 10, 2021 at 7:49 AM Segher Boessenkool > > wrote: > > > > > > On Thu, Sep 09, 2021 at 08:16:16AM +0200, Richard Biener wrote: > > > > > I think we should (longer term

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Fri, Sep 10, 2021 at 9:44 PM Hongtao Liu wrote: > > On Fri, Sep 10, 2021 at 9:32 PM Richard Biener > wrote: > > > > On September 10, 2021 3:27:09 PM GMT+02:00, Hongtao Liu > > wrote: > > >On Fri, Sep 10, 2021 at 9:16 PM Richard Biener via Gcc-patches > > > wrote: > > >> > > >> On Fri, Sep 10

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Qing Zhao via Gcc-patches
Hi, Christophe, Thanks for reporting the issue. Yes, the testing cases I added only got tested on aarch64 and x86_64 platforms, failures on other platforms are expected. I will take a look on this and to see how to resolve the testing issues. Qing > On Sep 10, 2021, at 3:47 AM, Christophe LY

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Qing Zhao via Gcc-patches
Hi, Thomas, Thanks for reporting the issue. > On Sep 10, 2021, at 4:40 AM, Thomas Schwinge wrote: > > Hi! > > On 2021-09-10T10:47:00+0200, Christophe LYON via Gcc-patches > wrote: >> On 10/09/2021 00:49, Qing Zhao via Gcc-patches wrote: >>> I just committed the following patch to gcc upstrea

Re: More aggressive threading causing loop-interchange-9.c regression

2021-09-10 Thread Michael Matz via Gcc-patches
Hello, On Fri, 10 Sep 2021, Aldy Hernandez via Gcc-patches wrote: > Like this? Yep, but I can't approve. Ciao, Michael.

Re: [COMMITTED][patch][version 9]add -ftrivial-auto-var-init and variable attribute "uninitialized" to gcc

2021-09-10 Thread Qing Zhao via Gcc-patches
Hi, Richard, Jose, Yes, we will try to come up with a patch to gcc-12/changes.html for this feature. Thanks. Qing > On Sep 10, 2021, at 8:46 AM, Jose E. Marchesi via Gcc-patches > wrote: > > > Hi Richard. > >> On Thu, 9 Sep 2021, Kees Cook wrote: >> >>> On Thu, Sep 09, 2021 at 10:49:11PM

Re: [PATCH] analyzer: Define INCLUDE_UNIQUE_PTR

2021-09-10 Thread David Malcolm via Gcc-patches
On Fri, 2021-09-10 at 09:01 +0100, Maxim Blinov wrote: > Un-break the build for AArch64 Darwin. Build currently fails with an > error very similar to pr82091: > > ``` > In file included from ../../../gcc-master-wip-apple- > si/gcc/analyzer/engine.cc:69: > In file included from > /Applications/Xcod

[PATCH v3 0/3] lower more cases of memcpy [PR102125]

2021-09-10 Thread Richard Earnshaw via Gcc-patches
Changes since version 2: patch 1 is reworked again. patch 2 is unchanged from v2, it's included here only for completeness. patch 3 is unchanged, it's included here only for completeness. - This short patch series is designed to address some more cases where we can usefully lower

[PATCH v3 1/3] rtl: directly handle MEM in gen_highpart [PR102125]

2021-09-10 Thread Richard Earnshaw via Gcc-patches
gen_lowpart_general handles forming a lowpart of a MEM by using adjust_address to rework and validate a new version of the MEM. Do the same for gen_highpart rather than calling simplify_gen_subreg for this case. gcc/ChangeLog: PR target/102125 * emit-rtl.c (gen_highpart): Use adj

[PATCH v3 2/3] arm: expand handling of movmisalign for DImode [PR102125]

2021-09-10 Thread Richard Earnshaw via Gcc-patches
DImode is currently handled only for machines with vector modes enabled, but this is unduly restrictive and is generally better done in core registers. gcc/ChangeLog: PR target/102125 * config/arm/arm.md (movmisaligndi): New define_expand. * config/arm/vec-common.md (movm

[PATCH v3 3/3] gimple: allow more folding of memcpy [PR102125]

2021-09-10 Thread Richard Earnshaw via Gcc-patches
The current restriction on folding memcpy to a single element of size MOVE_MAX is excessively cautious on most machines and limits some significant further optimizations. So relax the restriction provided the copy size does not exceed MOVE_MAX * MOVE_RATIO and that a SET insn exists for moving th

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread Segher Boessenkool
On Fri, Sep 10, 2021 at 12:53:37PM +0200, Richard Biener wrote: > On Fri, Sep 10, 2021 at 1:50 AM Segher Boessenkool > wrote: > > And many targets have strange rules for bit-strings in which modes can > > be used as bit-strings in which other modes, and at what offsets in > > which registers. Now

Re: [PATCH] Always default to DWARF2_DEBUG if not specified, warn about deprecated STABS

2021-09-10 Thread Michael Matz via Gcc-patches
Hello, On Fri, 10 Sep 2021, Richard Biener via Gcc-patches wrote: > diagnostic from the Ada frontend. The warnings are pruned from the > testsuite output via prune_gcc_output but somehow this doesn't work > for the tests in gfortran.dg/debug which are now failing with excess > errors. That seem

Re: [PATCH] Remove DARWIN_PREFER_DWARF and dead code

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 1:19 AM, Richard Biener via Gcc-patches wrote: This removes the always defined DARWIN_PREFER_DWARF and the code guarded by it being not defined, removing the possibility to default some i386 darwin configurations to STABS when it would not be defined. OK for trunk? Thanks, Richa

Re: [PATCH] Always default to DWARF2 debug for cygwin and mingw

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 12:38 AM, Richard Biener via Gcc-patches wrote: This removes the fallback to STABS as default for cygwin and mingw when the assembler does not support .secrel32 and the default is to emit 32bit code. Support for .secrel32 was added to binutils 2.16 released in 2005 so instead doc

Re: [PATCH] Fix SFmode subreg of DImode and TImode

2021-09-10 Thread Segher Boessenkool
On Fri, Sep 10, 2021 at 11:09:31AM +0800, Hongtao Liu wrote: > On Fri, Sep 10, 2021 at 7:49 AM Segher Boessenkool > wrote: > > way too long. But from that same history it follows that anything you > > do not super carefully (with testing everywhere) will cause some serious > Frankly, testing ever

Re: [PATCH, Fortran] Revert to non-multilib-specific ISO_Fortran_binding.h

2021-09-10 Thread Andreas Schwab
This misses the m68k extended real format. Andreas. * ISO_Fortran_binding.h (CFI_type_long_double) (CFI_type_long_double_Complex) [LDBL_MANT_DIG == 64 && LDBL_MIN_EXP == -16382 && LDBL_MAX_EXP == 16384]: Define. --- libgfortran/ISO_Fortran_binding.h | 7 +++ 1 file ch

Re: [PATCH] Default Alpha/VMS to DWARF2 debugging only

2021-09-10 Thread Douglas Rupp via Gcc-patches
Dwarf2 only works with an ancient GDB. VMS style was useful because it implemented enough to get a trace back. But as you say it’s dead, so no basis for objection. On Fri, Sep 10, 2021 at 6:47 AM Jeff Law wrote: > > > On 9/10/2021 12:52 AM, Richard Biener via Gcc-patches wrote: > > This changes

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Segher Boessenkool
On Fri, Sep 10, 2021 at 03:58:47PM +0200, Richard Biener wrote: > On September 10, 2021 3:30:10 PM GMT+02:00, Segher Boessenkool > wrote: > >TRULY_NOOP_TRUNCATION does not make sense to ask if changing mode class. > > OK, so there's a mode class comparison missing here which should be a better

Re: More aggressive threading causing loop-interchange-9.c regression

2021-09-10 Thread Jeff Law via Gcc-patches
On 9/10/2021 7:53 AM, Aldy Hernandez via Gcc-patches wrote: On 9/10/21 3:16 PM, Michael Matz wrote: Hi, On Fri, 10 Sep 2021, Aldy Hernandez via Gcc-patches wrote:   } + +  /* Threading through a non-empty latch would cause code to be added "through an *empty* latch".  The test in c

Go patch committed: Correct condition for calling memclrHasPointers

2021-09-10 Thread Ian Lance Taylor via Gcc-patches
This Go frontend patch corrects the condition under which we call memclrHasPointers. When compiling append(s, make([]typ, ln)...), where typ has a pointer, and the append fits within the existing capacity of s, the condition used to clear out the new elements was reversed. This fixes https://gola

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Richard Biener via Gcc-patches
On September 10, 2021 6:24:50 PM GMT+02:00, Segher Boessenkool wrote: >On Fri, Sep 10, 2021 at 03:58:47PM +0200, Richard Biener wrote: >> On September 10, 2021 3:30:10 PM GMT+02:00, Segher Boessenkool >> wrote: >> >TRULY_NOOP_TRUNCATION does not make sense to ask if changing mode class. >> >>

openmp: Implement OpenMP 5.1 atomics, so far for C only

2021-09-10 Thread Jakub Jelinek via Gcc-patches
Hi! This patch implements OpenMP 5.1 atomics (with clarifications from upcoming 5.2). The most important changes are that it is now possible to write (for C/C++, for Fortran it was possible before already) min/max atomics and more importantly compare and exchange in various forms. Also, acq_rel i

PING – [Patch] Fortran: Fix Bind(C) Array-Descriptor Conversion (Move to Front-End Code)

2021-09-10 Thread Tobias Burnus
Early PING for that patch. On 06.09.21 12:52, Tobias Burnus wrote: Hi all, gfortran's internal array descriptor (xgfc descriptor) and the descriptor used with BIND(C) (CFI descriptor, ISO_Fortran_binding.h of TS29113 / Fortran 2018) are different. Thus, when calling a BIND(C) procedure the gfc

[PATCH] MAINTAINERS: Adding myself to to DCO and write after approval

2021-09-10 Thread Petter Tomner via Gcc-patches
>From f75e52427846bc453544833b1d167f8568e7cfd8 Mon Sep 17 00:00:00 2001 From: Petter Tomner Date: Fri, 10 Sep 2021 21:37:00 +0200 Subject: [PATCH] MAINTAINERS: Adding myself to to DCO and write after approval 2020-09-10 Petter Tomner ChangeLog: * MAINTAINERS: Me added to DCO and

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Segher Boessenkool
On Fri, Sep 10, 2021 at 10:25:45PM +0800, Hongtao Liu wrote: > Updated patch. > > Bootstrapped and regtested on x86_64-linux-gnu{-m32,}, do I need to > run this patch on other targets machine, or the patch is supposed to > have minimal impact on other targets? > Then, ok for trunk? [-- Attac

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Segher Boessenkool
On Fri, Sep 10, 2021 at 08:36:12PM +0200, Richard Biener wrote: > On September 10, 2021 6:24:50 PM GMT+02:00, Segher Boessenkool > wrote: > >Yes, we should not call TRULY_NOOP_TRUNCATION_MODES_P for any random two > >modes: such a truncation needs to have a meaning at all, for the > >question to

Re: rs6000: add support for powerpc64le-unknown-freebsd

2021-09-10 Thread Piotr Kubaj via Gcc-patches
Hello again, it looks like one simple patch got left out by accident. Would it be possible for you to commit it? Thank you, Piotr Kubaj. On 20-12-28 06:37:23, Segher Boessenkool wrote: > On Mon, Dec 28, 2020 at 12:44:15PM +0100, Gerald Pfeifer wrote: > > On Wed, 16 Dec 2020, Segher Boessenkool

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Sat, Sep 11, 2021 at 5:21 AM Segher Boessenkool wrote: > > On Fri, Sep 10, 2021 at 10:25:45PM +0800, Hongtao Liu wrote: > > Updated patch. > > > > Bootstrapped and regtested on x86_64-linux-gnu{-m32,}, do I need to > > run this patch on other targets machine, or the patch is supposed to > >

Re: [PATCH 2/2] validate_subreg before call gen_lowpart to avoid ICE.

2021-09-10 Thread Hongtao Liu via Gcc-patches
On Sat, Sep 11, 2021 at 8:29 AM Hongtao Liu wrote: > > On Sat, Sep 11, 2021 at 5:21 AM Segher Boessenkool > wrote: > > > > On Fri, Sep 10, 2021 at 10:25:45PM +0800, Hongtao Liu wrote: > > > Updated patch. > > > > > > Bootstrapped and regtested on x86_64-linux-gnu{-m32,}, do I need to > > > run

[PATCH] libiberty: allow comments in option file

2021-09-10 Thread Hu Jialun
Enables putting line comments into option files included using '@'. Comments begin with a ';' followed by whitespace, and terminates at the end of the line. Backward compability should be satisfactory since ';' is a character disallowed in DOS filenames and rarely used in filenames or options any