Re: [PATCH] ifcvt.cc: Prevent excessive if-conversion for conditional moves

2023-01-11 Thread Robin Dapp via Gcc-patches
Hi, > On optimizing for speed, default_noce_conversion_profitable_p() allows > plenty of headroom, so this patch has little impact. > > Also, if the target-specific cost estimate is accurate or allows for > margins, the impact should be similarly small. I believe this part of ifcvt does/did not

Re: [committed] testsuite: Add testcases from PR108292 and PR108308

2023-01-11 Thread NightStrike via Gcc-patches
On Fri, Jan 6, 2023 at 4:56 AM Jakub Jelinek via Gcc-patches wrote: > --- gcc/testsuite/gcc.dg/pr108308.c.jj 2023-01-06 10:43:45.793009294 +0100 > +++ gcc/testsuite/gcc.dg/pr108308.c 2023-01-06 10:43:40.218090375 +0100 > @@ -0,0 +1,39 @@ > +/* PR target/108308 */ > +/* { dg-do run { target {

Re: [committed] testsuite: Add testcases from PR108292 and PR108308

2023-01-11 Thread Jakub Jelinek via Gcc-patches
On Wed, Jan 11, 2023 at 03:58:40AM -0500, NightStrike wrote: > On Fri, Jan 6, 2023 at 4:56 AM Jakub Jelinek via Gcc-patches > wrote: > > --- gcc/testsuite/gcc.dg/pr108308.c.jj 2023-01-06 10:43:45.793009294 +0100 > > +++ gcc/testsuite/gcc.dg/pr108308.c 2023-01-06 10:43:40.218090375 +0100 > > @

[PATCH] fortran: Fix up function types for realloc and sincos{,f,l} builtins [PR108349]

2023-01-11 Thread Jakub Jelinek via Gcc-patches
Hi! As reported in the PR, the FUNCTION_TYPE for __builtin_realloc in the Fortran FE is wrong since r0-100026-gb64fca63690ad which changed -  tmp = tree_cons (NULL_TREE, pvoid_type_node, void_list_node); -  tmp = tree_cons (NULL_TREE, size_type_node, tmp); -  ftype = build_function_type (pvoid_typ

Re: [PATCH] fortran: Fix up function types for realloc and sincos{,f,l} builtins [PR108349]

2023-01-11 Thread Tobias Burnus
Hi, On 11.01.23 10:18, Jakub Jelinek via Gcc-patches wrote: As reported in the PR, the FUNCTION_TYPE for __builtin_realloc in the Fortran FE is wrong since r0-100026-gb64fca63690ad [...] I went through all other changes from that commit and found that __builtin_sincos{,f,l} got broken as well, [

Re: [committed] testsuite: Add testcases from PR108292 and PR108308

2023-01-11 Thread NightStrike via Gcc-patches
On Wed, Jan 11, 2023 at 4:07 AM Jakub Jelinek wrote: > > On Wed, Jan 11, 2023 at 03:58:40AM -0500, NightStrike wrote: > > On Fri, Jan 6, 2023 at 4:56 AM Jakub Jelinek via Gcc-patches > > wrote: > > > --- gcc/testsuite/gcc.dg/pr108308.c.jj 2023-01-06 10:43:45.793009294 > > > +0100 > > > +++ gcc/

Re: [x86 PATCH] PR rtl-optimization/107991: peephole2 to tweak register allocation.

2023-01-11 Thread Uros Bizjak via Gcc-patches
On Tue, Jan 10, 2023 at 4:01 PM Roger Sayle wrote: > > > Hi Richard and Uros, > I believe I've managed to reduce a minimal test case that exhibits the > underlying > problem with reload. The following snippet when compiled on x86-64 with > -O2: > > void ext(int x); > void foo(int x, int y) { ext

Re: [committed] testsuite: Add testcases from PR108292 and PR108308

2023-01-11 Thread Jakub Jelinek via Gcc-patches
On Wed, Jan 11, 2023 at 04:27:11AM -0500, NightStrike wrote: > On Wed, Jan 11, 2023 at 4:07 AM Jakub Jelinek wrote: > > > > On Wed, Jan 11, 2023 at 03:58:40AM -0500, NightStrike wrote: > > > On Fri, Jan 6, 2023 at 4:56 AM Jakub Jelinek via Gcc-patches > > > wrote: > > > > --- gcc/testsuite/gcc.dg

[PATCH] c++: Avoid some false positive -Wfloat-conversion warnings with extended precision [PR108285]

2023-01-11 Thread Jakub Jelinek via Gcc-patches
Hi! On the following testcase trunk emits a false positive warning on ia32. convert_like_internal is there called with type of double and expr EXCESS_PRECISION_EXPR with float type with long double operand 2.L * (long double) x. Now, for the code generation we do the right thing, cp_convert to dou

Re: [PATCH 9/15] arm: Set again stack pointer as CFA reg when popping if necessary

2023-01-11 Thread Andrea Corallo via Gcc-patches
Richard Earnshaw writes: > On 09/01/2023 16:48, Richard Earnshaw via Gcc-patches wrote: >> On 09/01/2023 14:58, Andrea Corallo via Gcc-patches wrote: >>> Andrea Corallo via Gcc-patches writes: >>> Richard Earnshaw writes: > On 27/09/2022 16:24, Kyrylo Tkachov via Gcc-patches wrote

[Patch] Resolve bugzilla #108150 and #108192 for mingw

2023-01-11 Thread Jonathan Yong via Gcc-patches
Are the patches and changelogs OK?From 6edfba9e9a5f8fddc45d137b9f2d07c7f9065eaa Mon Sep 17 00:00:00 2001 From: Jonathan Yong <10wa...@gmail.com> Date: Sun, 8 Jan 2023 01:28:34 + Subject: [PATCH 1/2] PR c/108150 - Fix alignment test for Windows targets gcc/testsuite/ChangeLog: PR c/108150 *

[PATCH 10/15 V7] arm: Implement cortex-M return signing address codegen

2023-01-11 Thread Andrea Corallo via Gcc-patches
Richard Earnshaw writes: [...] > > Otherwise ok with that change. > > R. Minor respin of this patch addressing the suggestion to have 'use_return_insn' return zero when PAC is enabled. BR Andrea >From 0a894f73fc09be865b7a7cb205e871bf82f8abba Mon Sep 17 00:00:00 2001 From: Andrea Corallo D

[PATCH] Fix PR tree-optimization/108199

2023-01-11 Thread Eric Botcazou via Gcc-patches
Hi, this fixes the problematic interaction between bitfields, unions, SSO and SRA. Tested on x86-64/Linux and SPARC/Solaris, OK for all active branches? 2023-01-11 Eric Botcazou Andreas Krebbel PR tree-optimization/108199 * tree-sra.cc (sra_modify_expr): Deal w

Re: [committed] testsuite: Add testcases from PR108292 and PR108308

2023-01-11 Thread NightStrike via Gcc-patches
On Wed, Jan 11, 2023 at 4:43 AM Jakub Jelinek wrote: > > On Wed, Jan 11, 2023 at 04:27:11AM -0500, NightStrike wrote: > > On Wed, Jan 11, 2023 at 4:07 AM Jakub Jelinek wrote: > > > > > > On Wed, Jan 11, 2023 at 03:58:40AM -0500, NightStrike wrote: > > > > On Fri, Jan 6, 2023 at 4:56 AM Jakub Jeli

Re: [PATCH] libsanitizer/mips: always build with largefile support

2023-01-11 Thread YunQiang Su
Hans-Peter Nilsson 于2023年1月11日周三 08:53写道: > > On Fri, 6 Jan 2023, YunQiang Su wrote: > > > -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 is always used for mips > > when build libsanitizer in LLVM. Thus > >FIRST_32_SECOND_64((_MIPS_SIM == _ABIN32) ? 176 : 160, 216); > > instead of > >FIRST_32

[Committed] IBM zSystems: Use NAND instruction to implement bit not

2023-01-11 Thread Andreas Krebbel via Gcc-patches
Bootstrapped and regression tested on s390x. Committed to mainline. gcc/ChangeLog: * config/s390/s390.md (*not): New pattern. gcc/testsuite/ChangeLog: * gcc.target/s390/not.c: New test. --- gcc/config/s390/s390.md | 8 gcc/testsuite/gcc.target/s390/not.c

[PATCH] switch expansion: limit JT growth param values

2023-01-11 Thread Martin Liška
Currently, one can request a huge jump table creation which leads to a non-sensual huge output. Moreover, use auto_vec rather than a stack-allocated array. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin PR middle-end/107976 g

Re: [PATCH 10/15 V7] arm: Implement cortex-M return signing address codegen

2023-01-11 Thread Richard Earnshaw via Gcc-patches
On 11/01/2023 09:58, Andrea Corallo via Gcc-patches wrote: Richard Earnshaw writes: [...] Otherwise ok with that change. R. Minor respin of this patch addressing the suggestion to have 'use_return_insn' return zero when PAC is enabled. BR Andrea + /* Never use a return instruc

Re: [PATCH 1/15 V2] arm: Make mbranch-protection opts parsing common to AArch32/64

2023-01-11 Thread Richard Earnshaw via Gcc-patches
On 22/12/2022 17:04, Andrea Corallo via Gcc-patches wrote: Hi all, respinning this as a rebase was necessary, also now is setting 'aarch_enable_bti' to zero as default for arm as suggested during the review of 12/15. Best Regards Andrea gcc/ChangeLog: * common/config/aarch64

[PATCH] tree-optimization/108353 - copyprop iteration order

2023-01-11 Thread Richard Biener via Gcc-patches
After recent improvements to copyprop to catch more constants it shows that the current iteration order prefering forward progress over iterating doesn't make much sense for an SSA propagator. The following instead first iterates cycles which makes sure to not start with optimistically constant PH

[gcc-12 backport] strlen: do not use cond_expr for boundaries

2023-01-11 Thread Martin Liška
Tested, I'm going to push it. Martin PR tree-optimization/108137 gcc/ChangeLog: * tree-ssa-strlen.cc (get_range_strlen_phi): Reject anything different from INTEGER_CST. gcc/testsuite/ChangeLog: * gcc.dg/tree-ssa/pr108137.c: New test. (cherry picked from commit

Re: [PATCH] libgcc: Fix uninitialized RA signing on AArch64 [PR107678]

2023-01-11 Thread Martin Liška
On 1/10/23 19:12, Jakub Jelinek via Gcc-patches wrote: > Anyway, the sooner this makes it into gcc trunk, the better, it breaks quite > a lot of stuff. Yep, please, we're also waiting for this patch for pushing to our gcc13 package. Cheers, Martin

[PING^2] nvptx: Re-enable a number of test cases

2023-01-11 Thread Thomas Schwinge
Hi! Ping this whole series. Grüße Thomas On 2022-12-20T08:56:42+0100, I wrote: > Hi! > > Ping this whole series. > > > Grüße > Thomas > > > On 2022-12-02T13:03:06+0100, I wrote: >> Hi! >> >> I'm proposing to re-enable a number of test cases for nvptx. OK to push? >> >> >> Grüße >> Thomas -

[PING] nvptx: Make 'nvptx_uniform_warp_check' fit for non-full-warp execution (was: [committed][nvptx] Add uniform_warp_check insn)

2023-01-11 Thread Thomas Schwinge
Hi! Ping. Grüße Thomas On 2022-12-15T19:27:08+0100, I wrote: > Hi Tom! > > First "a bit" of context; skip to "the proposed patch" if you'd like to > see just that. > > > On 2022-02-01T19:31:27+0100, Tom de Vries via Gcc-patches > wrote: >> On a GT 1030, with driver version 470.94 and -mptx=

[PING] Add '-Wno-complain-wrong-lang', and use it in 'gcc/testsuite/lib/target-supports.exp:check_compile' and elsewhere

2023-01-11 Thread Thomas Schwinge
Hi! Ping. Grüße Thomas On 2022-12-16T15:10:12+0100, I wrote: > Hi! > > On 2022-12-15T16:17:05+0100, Jakub Jelinek wrote: >> On Thu, Dec 15, 2022 at 04:01:33PM +0100, Thomas Schwinge wrote: >>> Or, options are applicable to just one front end, and can just be a no-op >>> for others, for share

[PATCH] [RISCV] Add 'Zfa' extension according to riscv-isa-manual

2023-01-11 Thread jinma via Gcc-patches
From e4ce8e825c145d74e6b9827f972629548e39f118 Mon Sep 17 00:00:00 2001 From: Jin Ma Date: Wed, 11 Jan 2023 19:13:27 +0800 Subject: [PATCH] [RISCV] Add 'Zfa' extension according to riscv-isa-manual This patch adds the 'Zfa' extension for riscv, which is an implementation for unratified and unfroze

[PING] [PATCH 2/2] nvptx: Prevent emitting duplicate declarations for '__nvptx_stacks', '__nvptx_uni'

2023-01-11 Thread Thomas Schwinge
Hi! Ping. Grüße Thomas On 2022-12-19T21:40:07+0100, I wrote: > As I have reported to Nvidia in 2022-12-01 'NVIDIA Incident Report (3891704): > ptxas: Duplicate declaration error: "cannot be resolved by a '.static'"', > 'ptxas' has an inscrutable error mode for duplicate declarations: > >

[PING^3] nvptx: stack size limits are relevant for execution only (was: [PATCH, testsuite] Add effective target stack_size)

2023-01-11 Thread Thomas Schwinge
Hi! Ping. Grüße Thomas On 2022-12-20T08:55:08+0100, I wrote: > Hi! > > Ping. > > > Grüße > Thomas > > > On 2022-11-25T12:09:36+0100, I wrote: >> Hi! >> >> Ping. >> >> >> Grüße >> Thomas >> >> >> On 2022-11-08T21:29:49+0100, I wrote: >>> Hi! >>> >>> On 2017-06-09T16:24:30+0200, Tom de Vries

[PING^2] nvptx: Support global constructors/destructors via 'collect2'

2023-01-11 Thread Thomas Schwinge
Hi! Ping. Grüße Thomas On 2022-12-20T09:03:51+0100, I wrote: > Hi! > > Ping. > > > Minor change in the attached > "nvptx: Support global constructors/destructors via 'collect2'": for > 'atexit', add '#include ' to 'libgcc/config/nvptx/crt0.c'. > > > Grüße > Thomas > > > On 2022-12-02T14:35:3

[PING] nvptx: Support global constructors/destructors via 'collect2' for offloading (was: nvptx: Support global constructors/destructors via 'collect2')

2023-01-11 Thread Thomas Schwinge
Hi! Ping. Grüße Thomas On 2022-12-23T14:37:47+0100, I wrote: > Hi! > > On 2022-12-23T14:35:16+0100, I wrote: >> On 2022-12-02T14:35:35+0100, I wrote: >>> On 2022-12-01T22:13:38+0100, I wrote: I'm working on support for global constructors/destructors with GCC/nvptx >>> >>> See "nvpt

[PATCH] tree-optimization/108352 - FSM threads creating irreducible loops

2023-01-11 Thread Richard Biener via Gcc-patches
The following relaxes a heuristic that prevents creating irreducible loops from FSM threads not covering multi-way branches. Instead of allowing threads that adhere to && (n_insns * (unsigned) param_fsm_scale_path_stmts > (m_path.length () * (unsigned) param_fsm_scale

Re: [PATCH] libgcc: Fix uninitialized RA signing on AArch64 [PR107678]

2023-01-11 Thread Wilco Dijkstra via Gcc-patches
Hi, > On 1/10/23 19:12, Jakub Jelinek via Gcc-patches wrote: >> Anyway, the sooner this makes it into gcc trunk, the better, it breaks quite >> a lot of stuff. > > Yep, please, we're also waiting for this patch for pushing to our gcc13 > package. Well I'm waiting for an OK from a maintainer... I

Re: [PATCH] Fix PR tree-optimization/108199

2023-01-11 Thread Richard Biener via Gcc-patches
On Wed, Jan 11, 2023 at 11:00 AM Eric Botcazou via Gcc-patches wrote: > > Hi, > > this fixes the problematic interaction between bitfields, unions, SSO and SRA. > > Tested on x86-64/Linux and SPARC/Solaris, OK for all active branches? OK. Thanks, Richard. > > 2023-01-11 Eric Botcazou >

Re: [PATCH] switch expansion: limit JT growth param values

2023-01-11 Thread Richard Biener via Gcc-patches
On Wed, Jan 11, 2023 at 11:31 AM Martin Liška wrote: > > Currently, one can request a huge jump table creation which > leads to a non-sensual huge output. Moreover, use auto_vec rather > than a stack-allocated array. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Re

[PING] nvptx: '-mframe-malloc-threshold', '-Wframe-malloc-threshold' (was: Handling of large stack objects in GPU code generation -- maybe transform into heap allocation?)

2023-01-11 Thread Thomas Schwinge
Hi! Ping -- the '-mframe-malloc-threshold' idea, at least. Note that while this issue originally did pop up for Fortran I/O, it's likewise relevant for other functions that maintain big frames, for example in newlib: libc/string/libc_a-memmem.o:.local .align 16 .b8 %frame_ar[2064]; libc/

Re: [PATCH] tree-optimization/107767 - not profitable switch conversion

2023-01-11 Thread Martin Liška
On 1/9/23 12:09, Richard Biener wrote: > |Martin, OK with you?| Yes, thanks for handling that. Martin

Re: [committed] testsuite: Add testcases from PR108292 and PR108308

2023-01-11 Thread Jakub Jelinek via Gcc-patches
On Wed, Jan 11, 2023 at 05:10:50AM -0500, NightStrike wrote: > Ok, then: > > /* { dg-do run { target { { ilp32 || lp64 } || llp64 } } } */ > > or even: > > /* { dg-do run { target { ! int16 } } } */ > > Though I'd point out that in your original message, you only cared > about the "important ta

Re: [committed] testsuite: Add testcases from PR108292 and PR108308

2023-01-11 Thread NightStrike via Gcc-patches
On Wed, Jan 11, 2023 at 7:14 AM Jakub Jelinek wrote: > I've committed following after regtesting it on x86_64-linux and i686-linux: ... > +/* { dg-do run { target int32 } } */ Ah, I didn't realize you meant literally int32. I didn't see that as a choice here: https://gcc.gnu.org/onlinedocs/gcci

Re: [PING] Add '-Wno-complain-wrong-lang', and use it in 'gcc/testsuite/lib/target-supports.exp:check_compile' and elsewhere

2023-01-11 Thread Jakub Jelinek via Gcc-patches
Hi! On Wed, Jan 11, 2023 at 12:41:06PM +0100, Thomas Schwinge wrote: I think this should be reviewed by Joseph as option handling maintainer. > @@ -8896,6 +8897,13 @@ programs. > Warn for variables that might be changed by @code{longjmp} or > @code{vfork}. This warning is also enabled by @opt

Re: [PATCH] rs6000: Make P10_FUSION honour tuning setting

2023-01-11 Thread Kewen.Lin via Gcc-patches
on 2023/1/6 17:28, Kewen.Lin via Gcc-patches wrote: > Hi Pat, > > on 2023/1/6 03:30, Pat Haugen wrote: >> On 1/4/23 3:20 AM, Kewen.Lin via Gcc-patches wrote: >>> diff --git a/gcc/config/rs6000/rs6000.cc b/gcc/config/rs6000/rs6000.cc >>> index 88c865b6b4b..6fa084c0807 100644 >>> --- a/gcc/config/rs

[PATCH, committed] rs6000/test: Make ppc-fortran.exp only available for PowerPC target

2023-01-11 Thread Kewen.Lin via Gcc-patches
Hi, When testing one patch which adds a fortran test case into test bucket powerpc/ppc-fortran/, I found one unexpected failure on a non-PowerPC target. It's due to that ppc-fortran.exp does not exit early if the testing target isn't a PowerPC target. This patch is to make it exit immediately if

[ping2][PATCH 0/2] __bos and flex arrays

2023-01-11 Thread Siddhesh Poyarekar
Ping! On 2022-12-21 17:25, Siddhesh Poyarekar wrote: Hi, The first patch in the series is just a minor test cleanup that I did to make sure all tests in a test case run (instead of aborting at first failure) and print the ones that failed. The second patch is the actual fix. The patch intends

[PATCH] rs6000: Imply VSX early to adopt some checkings on conflict [PR108240]

2023-01-11 Thread Kewen.Lin via Gcc-patches
Hi, As PR108240 shows, some options like -mmodulo can enable some flags implicitly including OPTION_MASK_VSX. But the enabled flag can conflict with some existing setting like soft float, it would result in some unexpected cases and consequent ICE. Actually there are already some checkings for VS

Re: [PATCH v3] Add pattern to convert vector shift + bitwise and + multiply to vector compare in some cases.

2023-01-11 Thread Manolis Tsamis
Hi Richard and Tamar, I just wanted to ping you about this patch. Is there a chance to get this into GCC13? Thanks, Manolis On Tue, Dec 20, 2022 at 2:31 PM Manolis Tsamis wrote: > > On Tue, Dec 20, 2022 at 2:23 PM Manolis Tsamis > wrote: > > > > When using SWAR (SIMD in a register) techniques

[PATCH v3 2/2] aarch64: Fix bit-field alignment in param passing [PR105549]

2023-01-11 Thread Christophe Lyon via Gcc-patches
While working on enabling DFP for AArch64, I noticed new failures in gcc.dg/compat/struct-layout-1.exp (t028) which were not actually caused by DFP types handling. These tests are generated during 'make check' and enabling DFP made generation different (not sure if new non-DFP tests are generated,

[PATCH v3 1/2] aarch64: fix warning emission for ABI break since GCC 9.1

2023-01-11 Thread Christophe Lyon via Gcc-patches
While looking at PR 105549, which is about fixing the ABI break introduced in GCC 9.1 in parameter alignment with bit-fields, we noticed that the GCC 9.1 warning is not emitted in all the cases where it should be. This patch fixes that and the next patch in the series fixes the GCC 9.1 break. We

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Gerald Pfeifer
On Thu, 5 Jan 2023, Segher Boessenkool wrote: > Happy new year everyone. > > Is this patch okay to commit? >From a wwwdocs perspective, yes. Are you also *asking* from an architectural/"strategic" perspective, or simply *informing*? :-) The former I cannot approve, the latter I certainly can

Re: [PATCH,WWWDOCS] htdocs: news: GCC BPF in Compiler Explorer

2023-01-11 Thread Gerald Pfeifer
On Fri, 23 Dec 2022, Jose E. Marchesi via Gcc-patches wrote: > This patch adds an entry to the News section in index.html, announcing > the availability of a nightly build of bpf-unknown-none-gcc. Nice! > +https://godbolt.org";>GCC BPF in Compiler > Explorer > + [2022-12-23] > +Support for a

[PATCH 2/2 v2] arm: Add support for MVE Tail-Predicated Low Overhead Loops

2023-01-11 Thread Stam Markianos-Wright via Gcc-patches
-  Respin of the below patch - In this 2/2 patch, from v1 to v2 I have: * Removed the modification the interface of the doloop_end target-insn (so I no longer need to touch any other target backends) * Added more modes to `arm_get_required_vpr_reg` to make it flexible between searching

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Richard Biener via Gcc-patches
On Wed, Jan 11, 2023 at 3:22 PM Gerald Pfeifer wrote: > > On Thu, 5 Jan 2023, Segher Boessenkool wrote: > > Happy new year everyone. > > > > Is this patch okay to commit? > > From a wwwdocs perspective, yes. > > Are you also *asking* from an architectural/"strategic" perspective, > or simply *info

[ping][PATCH 1/1] docs: Add link to gmplib.org

2023-01-11 Thread Benson Muite via Gcc-patches
Improvement to documentation from a new contributor without commit rights. On 1/5/23 06:38, Benson Muite wrote: > Link is missing from install documentation > > Signed-off-by: Benson Muite > --- > gcc/doc/install.texi | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/g

Re: [PATCH 12/15 V5] arm: implement bti injection

2023-01-11 Thread Richard Earnshaw via Gcc-patches
On 22/12/2022 17:13, Andrea Corallo via Gcc-patches wrote: Richard Earnshaw writes: On 14/12/2022 17:00, Richard Earnshaw via Gcc-patches wrote: On 14/12/2022 16:40, Andrea Corallo via Gcc-patches wrote: Hi Richard, thanks for reviewing. Richard Earnshaw writes: On 28/10/2022 17:40,

Re: PING: New reg note REG_CFA_NORESTORE

2023-01-11 Thread Andreas Krebbel via Gcc-patches
On 12/27/22 19:23, Jeff Law wrote: > > > On 12/13/22 01:55, Andreas Krebbel via Gcc-patches wrote: >> Hi, >> >> I need a way to save registers on the stack and generate proper CFI for it. >> Since I do not intend to >> restore them I needed a way to tell the CFI generation step about it: >> >> h

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Segher Boessenkool
On Wed, Jan 11, 2023 at 03:34:45PM +0100, Richard Biener wrote: > On Wed, Jan 11, 2023 at 3:22 PM Gerald Pfeifer wrote: > > > > On Thu, 5 Jan 2023, Segher Boessenkool wrote: > > > Happy new year everyone. > > > > > > Is this patch okay to commit? > > > > From a wwwdocs perspective, yes. > > > > Ar

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Richard Biener via Gcc-patches
> Am 11.01.2023 um 16:17 schrieb Segher Boessenkool > : > > On Wed, Jan 11, 2023 at 03:34:45PM +0100, Richard Biener wrote: >>> On Wed, Jan 11, 2023 at 3:22 PM Gerald Pfeifer wrote: >>> >>> On Thu, 5 Jan 2023, Segher Boessenkool wrote: Happy new year everyone. Is this patch

RE: [GCC][PATCH v2] arm: Add cde feature support for Cortex-M55 CPU.

2023-01-11 Thread Srinath Parvathaneni via Gcc-patches
Ping!! - From: Srinath Parvathaneni Sent: Tuesday, December 6, 2022 11:32 AM To: gcc-patches@gcc.gnu.org; Richard Earnshaw Cc: Christophe Lyon Subject: Re: [GCC][PATCH v2] arm: Add cde feature support for Cortex-M55 CPU. Ping!! __

[PATCH] c++: Avoid incorrect shortening of divisions [PR108365]

2023-01-11 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase is miscompiled, because we shorten the division in a case where it should not be shortened. Divisions (and modulos) can be shortened if it is unsigned division/modulo, or if it is signed division/modulo where we can prove the dividend will not be the minimum signed value

[PATCH] c: Don't emit DEBUG_BEGIN_STMTs for K&R function argument declarations [PR105972]

2023-01-11 Thread Jakub Jelinek via Gcc-patches
Hi! K&R function parameter declarations are handled by calling recursively c_parser_declaration_or_fndef in a loop, where each such call will add_debug_begin_stmt at the start. Now, if the K&R function definition is not a nested function, building_stmt_list_p () is false and so we don't emit the D

[OG12][committed] amdgcn, libgomp: custom USM allocator

2023-01-11 Thread Andrew Stubbs
This patch fixes a runtime issue I encountered with the AMD GCN Unified Shared Memory implementation. We were using regular malloc'd memory configured into USM mode, but there were random intermittent crashes. I can't be completely sure, but my best guess is that the HSA driver is using malloc

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Segher Boessenkool
On Wed, Jan 11, 2023 at 05:27:36PM +0100, Richard Biener wrote: > > Am 11.01.2023 um 16:17 schrieb Segher Boessenkool > > : > >> Note this is more info for port maintainers not for users and > >> changes.html is for users. > > > > And users will notice some ports will have to be removed, because

Re: [PATCH] [PR40457] [arm] expand SI-aligned movdi into pair of movsi

2023-01-11 Thread Richard Earnshaw via Gcc-patches
On 02/12/2022 09:29, Alexandre Oliva via Gcc-patches wrote: When expanding a misaligned DImode move, emit aligned SImode moves if the parts are sufficiently aligned. This enables neighboring stores to be peephole-combined into stm, as expected by the PR40457 testcase, even after SLP vectoriz

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Richard Biener via Gcc-patches
> Am 11.01.2023 um 19:34 schrieb Segher Boessenkool > : > > On Wed, Jan 11, 2023 at 05:27:36PM +0100, Richard Biener wrote: Am 11.01.2023 um 16:17 schrieb Segher Boessenkool : > Note this is more info for port maintainers not for users and > changes.html is for users. >>>

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Paul Koning via Gcc-patches
> On Jan 11, 2023, at 1:32 PM, Segher Boessenkool > wrote: > > On Wed, Jan 11, 2023 at 05:27:36PM +0100, Richard Biener wrote: >>> Am 11.01.2023 um 16:17 schrieb Segher Boessenkool >>> : Note this is more info for port maintainers not for users and changes.html is for users. >>> >

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Segher Boessenkool
On Wed, Jan 11, 2023 at 07:39:29PM +0100, Richard Biener wrote: > Like if they cannot even build their target libraries aka their build will > fail. It would be nice to identify those and, say, make at least -mlra > available to all ports that currently do not have a way to enable LRA? It is up

Re: [PATCH] preprocessor: Don't register pragmas in directives-only mode [PR108244]

2023-01-11 Thread Jakub Jelinek via Gcc-patches
On Fri, Dec 30, 2022 at 12:21:37PM -0500, Lewis Hyatt via Gcc-patches wrote: > libcpp's directives-only mode does not expect deferred pragmas to be > registered, but to date the c-family registration process has not checked for > this case. That issue became more visible since r13-1544, which added

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Segher Boessenkool
On Wed, Jan 11, 2023 at 01:42:22PM -0500, Paul Koning wrote: > Or, as in my case, because building with LRA as the default triggers an ICE > that I don't understand. I posted a note to the GCC list about what I saw, > but have received no reaction.

Re: [PATCH 2/3] Make __float128 use the _Float128 type, PR target/107299

2023-01-11 Thread Michael Meissner via Gcc-patches
On Tue, Nov 01, 2022 at 10:42:30PM -0400, Michael Meissner wrote: > This patch fixes the issue that GCC cannot build when the default long double > is IEEE 128-bit. It fails in building libgcc, specifically when it is trying > to buld the __mulkc3 function in libgcc. It is failing in > gimple-ra

Re: [PATCH] c: Don't emit DEBUG_BEGIN_STMTs for K&R function argument declarations [PR105972]

2023-01-11 Thread Joseph Myers
On Wed, 11 Jan 2023, Jakub Jelinek via Gcc-patches wrote: > The following patch ensures we don't emit any such DEBUG_BEGIN_STMTs for the > K&R function parameter declarations even in nested functions. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2023-01-11 Jakub

Re: [RFC/PATCH] Remove the workaround for _Float128 precision [PR107299]

2023-01-11 Thread Michael Meissner via Gcc-patches
On Tue, Jan 10, 2023 at 07:23:23PM +0100, Jakub Jelinek wrote: > On Mon, Jan 09, 2023 at 10:21:52PM -0500, Michael Meissner wrote: > > I had the patches to change the precision to 128, and I just ran them. C > > and > > C++ do not seem to be bothered by changing the precision to 128 (once I got

[committed] analyzer: fix leak false positives on "*UNKNOWN = PTR; " [PR108252]

2023-01-11 Thread David Malcolm via Gcc-patches
PR analyzer/108252 reports a false positive from -Wanalyzer-malloc-leak on code like this: *ptr_ptr = strdup(EXPR); where ptr_ptr is an UNKNOWN_VALUE. When we handle: *UNKNOWN = PTR; store::set_value normally marks *PTR as having escaped, and this means we don't report PTR as leaking when th

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Paul Koning via Gcc-patches
> On Jan 11, 2023, at 2:28 PM, Segher Boessenkool > wrote: > > On Wed, Jan 11, 2023 at 01:42:22PM -0500, Paul Koning wrote: >> Or, as in my case, because building with LRA as the default triggers an ICE >> that I don't understand. I posted a note to the GCC list about what I saw, >> but ha

Re: [PATCH,WWWDOCS] htdocs: add an Atom feed for GCC news

2023-01-11 Thread Thomas Schwinge
Hi! On 2022-12-23T10:50:13+0100, "Jose E. Marchesi via Gcc-patches" wrote: > This patch adds an Atom feed for GCC news, which can then be easily > aggregated in other sites, such as the GNU planet > (https://planet.gnu.org). > > The feed lives in a file news.xml, and this patch initializes it wi

Re: [PATCH,WWWDOCS] htdocs: rotate news

2023-01-11 Thread Gerald Pfeifer
On Fri, 23 Dec 2022, Jose E. Marchesi via Gcc-patches wrote: > htdocs/index.html | 24 > htdocs/news.html | 24 > 2 files changed, 24 insertions(+), 24 deletions(-) Okay, thank you. And you can consider this kind of change preapproved. Or fallin

[committed] wwwdocs: gcc-8: Properly spell "command-line option"

2023-01-11 Thread Gerald Pfeifer
On the way add some missing "the"s. Pushed. Gerald --- htdocs/gcc-8/changes.html | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/htdocs/gcc-8/changes.html b/htdocs/gcc-8/changes.html index 73ccd07d..c329a509 100644 --- a/htdocs/gcc-8/changes.html +++ b/htdocs/gcc-8/ch

Re: [PATCH,WWWDOCS] htdocs: add an Atom feed for GCC news

2023-01-11 Thread Jose E. Marchesi via Gcc-patches
> Hi! > > On 2022-12-23T10:50:13+0100, "Jose E. Marchesi via Gcc-patches" > wrote: >> This patch adds an Atom feed for GCC news, which can then be easily >> aggregated in other sites, such as the GNU planet >> (https://planet.gnu.org). >> >> The feed lives in a file news.xml, and this patch ini

Re: [PATCH,WWWDOCS] htdocs: add an Atom feed for GCC news

2023-01-11 Thread Jose E. Marchesi via Gcc-patches
>> Hi! >> >> On 2022-12-23T10:50:13+0100, "Jose E. Marchesi via Gcc-patches" >> wrote: >>> This patch adds an Atom feed for GCC news, which can then be easily >>> aggregated in other sites, such as the GNU planet >>> (https://planet.gnu.org). >>> >>> The feed lives in a file news.xml, and this

[committed] config-list.mk: Remove obsolete FreeBSD targets

2023-01-11 Thread Gerald Pfeifer
ia64-freebsd is officially dead, and sparc64-freebsd has not been able to build GCC for half a dozen years (or so) and is essentially end of life. The default per gcc/config/i386/freebsd.h has been i586 for a while, so i486-freebsd can go as well. (We still have i686-freebsd.) Pushed, obvious rul

Re: [RFA] choosing __platform_wait_t on targets without lock-free 64 atomics

2023-01-11 Thread Thomas Rodgers via Gcc-patches
I agree with this change. On Thu, Jan 5, 2023 at 4:22 PM Jonathan Wakely wrote: > How about this? > > I don't think we should worry about targets without atomic int, so don't > bother using types smaller than int. > > > -- >8 -- > > For non-futex targets the __platform_wait_t type is currently u

[PATCH] gimple-fold.h: Add missing gimple-iterator.h

2023-01-11 Thread Palmer Dabbelt
As of 6f5b06032eb ("Finish gimple_build API enhancement") gimple-fold.h uses some of the declarations from gimple-iterator.h, which causes issues when building Linux's stackprotector plugin. gcc/ChangeLog: * gimple-fold.h: Add gimple-iterator.h include. --- I'm not sure if this should i

[PATCH v2] [RISCV] Add 'Zfa' extension according to riscv-isa-manual

2023-01-11 Thread jinma via Gcc-patches
From e4ce8e825c145d74e6b9827f972629548e39f118 Mon Sep 17 00:00:00 2001 From: Jin Ma Date: Wed, 11 Jan 2023 19:13:27 +0800 Subject: [PATCH] [RISCV] Add 'Zfa' extension according to riscv-isa-manual From e4ce8e825c145d74e6b9827f972629548e39f118 Mon Sep 17 00:00:00 2001 From: Jin Ma Date: Wed, 11 J

Re: [PING] nvptx: '-mframe-malloc-threshold', '-Wframe-malloc-threshold' (was: Handling of large stack objects in GPU code generation -- maybe transform into heap allocation?)

2023-01-11 Thread Jerry D via Gcc-patches
On 1/11/23 4:06 AM, Thomas Schwinge wrote: Hi! Ping -- the '-mframe-malloc-threshold' idea, at least. Note that while this issue originally did pop up for Fortran I/O, it's likewise relevant for other functions that maintain big frames, for example in newlib: libc/string/libc_a-memmem.o:.

Re: [PATCH] ifcvt.cc: Prevent excessive if-conversion for conditional moves

2023-01-11 Thread Takayuki 'January June' Suwa via Gcc-patches
On 2023/01/11 17:02, Robin Dapp wrote: > Hi, Hi! > >> On optimizing for speed, default_noce_conversion_profitable_p() allows >> plenty of headroom, so this patch has little impact. >> >> Also, if the target-specific cost estimate is accurate or allows for >> margins, the impact should be similar

Just test mailbox

2023-01-11 Thread ijinma--- via Gcc-patches
Sorry, I just want to test whether my mailbox function is correct. Please ignore it. Thank you.

Re: Just test mailbox

2023-01-11 Thread ijinma--- via Gcc-patches
new one. Replied Message | From | iji...@yeah.net | | Date | 01/12/2023 11:39 | | To | gcc-patches | | Cc | | | Subject | Just test mailbox | Sorry, I just want to test whether my mailbox function is correct. Please ignore it. Thank you.

Re: Just test mailbox

2023-01-11 Thread ijinma--- via Gcc-patches
one more. Replied Message | From | iji...@yeah.net | | Date | 01/12/2023 12:06 | | To | gcc-patches、MarJin | | Cc | | | Subject | Re: Just test mailbox | new one. Replied Message | From | iji...@yeah.net | | Date | 01/12/2023 11:39 | | To | gcc-patches | | Cc | | | Subject

Re: [PATCH v2] [RISCV] Add 'Zfa' extension according to riscv-isa-manual

2023-01-11 Thread jinma via Gcc-patches
From e4ce8e825c145d74e6b9827f972629548e39f118 Mon Sep 17 00:00:00 2001 From: Jin Ma Date: Wed, 11 Jan 2023 19:13:27 +0800 Subject: [PATCH] [RISCV] Add 'Zfa' extension according to riscv-isa-manual This patch adds the 'Zfa' extension for riscv, which is based on: ( https://github.com/riscv/riscv-i

[PATCH 1/2] xtensa: Tune "*btrue" insn pattern

2023-01-11 Thread Takayuki 'January June' Suwa via Gcc-patches
This branch instruction has short encoding if EQ/NE comparison against immediate zero when the Code Density Option is enabled, but its "length" attribute was only for normal encoding. This patch fixes it. This patch also prevents undesireable replacement the comparison immediate zero of the instr

[PATCH 2/2] xtensa: Optimize ctzsi2 and ffssi2 a bit

2023-01-11 Thread Takayuki 'January June' Suwa via Gcc-patches
This patch saves one byte when the Code Density Option is enabled, gcc/ChangeLog: * config/xtensa/xtensa.md (ctzsi2, ffssi2): Rearrange the emitting codes. --- gcc/config/xtensa/xtensa.md | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/gcc/config/xtens

Re: [PATCH v2] [RISCV] Add 'Zfa' extension according to riscv-isa-manual

2023-01-11 Thread jinma via Gcc-patches
From 4ee11f99d23d39d55bdadd86699ac35a60c79705 Mon Sep 17 00:00:00 2001 From: Jin Ma Date: Thu, 12 Jan 2023 12:51:37 +0800 Subject: [PATCH v2] [RISCV] Add 'Zfa' extension according to riscv-isa-manual This patch adds the 'Zfa' extension for riscv, which is based on: ( https://github.com/riscv/risc

Re: [PATCH v2] [RISCV] Add 'Zfa' extension according to riscv-isa-manual

2023-01-11 Thread jinma via Gcc-patches
I am very sorry. There seems to be some unknown problems in my email, which caused a problem with the format of the patch. I will deal with it as soon as possible. I am very sorry for the trouble.

[PATCH v3] [RISCV] Add 'Zfa' extension according to riscv-isa-manual

2023-01-11 Thread jinma via Gcc-patches
From 4ee11f99d23d39d55bdadd86699ac35a60c79705 Mon Sep 17 00:00:00 2001 In-Reply-To: <77a18666-f71d-48e2-a502-a879b3eb6ccf.ji...@linux.alibaba.com> References: <77a18666-f71d-48e2-a502-a879b3eb6ccf.ji...@linux.alibaba.com> From: Jin Ma Date: Thu, 12 Jan 2023 12:51:37 +0800 Subject: [PATCH v3] [RISC

Re: libstdc++: Fix deadlock in debug iterator increment [PR108288]

2023-01-11 Thread François Dumont via Gcc-patches
Small update for an obvious compilation issue and to review new test case that could have lead to an infinite loop if the increment issue was not detected. I also forgot to ask if there is more chance for the instantiation to be elided when it is implemented like in the _Safe_local_iterator:

[PATCH v4] [RISCV] Add 'Zfa' extension according to riscv-isa-manual

2023-01-11 Thread Jin Ma via Gcc-patches
This patch adds the 'Zfa' extension for riscv, which is based on: ( https://github.com/riscv/riscv-isa-manual/commit/d74d99e22d5f68832f70982d867614e2149a3bd7 ) latest 'Zfa' change on the master branch of the RISC-V ISA Manual as of this writing. The Wiki Page (details): ( https://github.com/a4lg

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Richard Biener via Gcc-patches
On Wed, Jan 11, 2023 at 8:09 PM Segher Boessenkool wrote: > > On Wed, Jan 11, 2023 at 07:39:29PM +0100, Richard Biener wrote: > > Like if they cannot even build their target libraries aka their build will > > fail. It would be nice to identify those and, say, make at least -mlra > > available t

Re: [PATCH] wwwdocs: Note that old reload is deprecated

2023-01-11 Thread Richard Biener via Gcc-patches
On Thu, Jan 12, 2023 at 8:44 AM Richard Biener wrote: > > On Wed, Jan 11, 2023 at 8:09 PM Segher Boessenkool > wrote: > > > > On Wed, Jan 11, 2023 at 07:39:29PM +0100, Richard Biener wrote: > > > Like if they cannot even build their target libraries aka their build > > > will fail. It would be

Re: [PATCH] gimple-fold.h: Add missing gimple-iterator.h

2023-01-11 Thread Richard Biener via Gcc-patches
On Thu, Jan 12, 2023 at 2:46 AM Palmer Dabbelt wrote: > > As of 6f5b06032eb ("Finish gimple_build API enhancement") gimple-fold.h > uses some of the declarations from gimple-iterator.h, which causes > issues when building Linux's stackprotector plugin. > > gcc/ChangeLog: > > * gimple-fold.