Re: arm: Make arm_noce_conversion_profitable_p call default hook [PR 116444]

2024-10-06 Thread Ramana Radhakrishnan
On Fri, Oct 4, 2024 at 9:25 PM Andre Vieira (lists) wrote: > > Hi, > > The patch for 'arm: Fix missed CE optimization for armv8.1-m.main [PR > 116444]' introduced regressions with arm targets that used 'noce' before. > This is because it would approve all noce optimisations without using > the def

Re: [PING] [PATCH] i386: Implement Thread Local Storage on Windows

2024-10-06 Thread Julian Waters
Hi Eric, Thanks for getting back to me, I wasn't actually asking if it was ok (The ping was poorly worded, sorry), more trying to draw attention to the patch. The linker bug blocking this patch has actually already been fixed, see https://github.com/bminor/binutils-gdb/commit/72cd2c70977943054ff78

[PING] [PATCH] i386: Implement Thread Local Storage on Windows

2024-10-06 Thread Julian Waters
Hi all, Pinging https://gcc.gnu.org/pipermail/gcc-patches/2024-September/662860.html as it has been buried under several other patches. Is the patch ok for master? best regards, Julian

Re: [PING] [PATCH] i386: Implement Thread Local Storage on Windows

2024-10-06 Thread Eric Botcazou
> Pinging https://gcc.gnu.org/pipermail/gcc-patches/2024-September/662860.html > as it has been buried under several other patches. Is the patch ok for > master? No, you should modify it along the way I suggested privately, and a blocker is the missing support in the linker AFAICS. -- Eric Botc

Re: [PATCH 3/3] phiprop: VOP phi confuses phiprop [PR116824]

2024-10-06 Thread Andrew Pinski
On Wed, Sep 25, 2024 at 1:38 AM Richard Biener wrote: > > On Tue, Sep 24, 2024 at 6:14 PM Andrew Pinski > wrote: > > > > Another small phiprop improvement, in some cases > > we could have a vop defining statement be a phi which might > > be the same bb as the load happens. This is ok since the p

Re: [PATCH 3/3] gimple: Add gimple_with_undefined_signed_overflow and use it [PR111276]

2024-10-06 Thread Andrew Pinski
On Fri, Oct 4, 2024 at 12:08 AM Richard Biener wrote: > > On Thu, Oct 3, 2024 at 6:09 PM Andrew Pinski wrote: > > > > While looking into the ifcombine, I noticed that rewrite_to_defined_overflow > > was rewriting already defined code. In the previous attempt at fixing this, > > the review mention

[to-be-committed][V2][RISC-V] Add splitters to restore condops generation after recent phiopt changes

2024-10-06 Thread Jeff Law
V2: Fix typo in ChangeLog. Remove now extraneous comment in cset-sext.c. Throttle back branch cost to 1 in various tests -- Andrew P's recent improvements to phiopt regressed on the riscv testsuite. Essentially the new code presented to the RTL optimizers is straightline code rather tha

[committed] hppa: Use stack slot SP-40 to copy between integer and floating-point registers

2024-10-06 Thread John David Anglin
Tested on hppa64-hp-hpux11.11 and hppa-unknown-linux-gnu. Committed to trunk. Dave --- hppa: Use stack slot SP-40 to copy between integer and floating-point registers 2024-10-06 John David Anglin gcc/ChangeLog: * config/pa/pa-64.h (PA_SECONDARY_MEMORY_NEEDED): Define to fal

Re: [PATCH v2] Add new warning Wmissing-designated-initializers [PR39589]

2024-10-06 Thread Peter Frost
Ping https://gcc.gnu.org/pipermail/gcc-patches/2024-September/662590.html

testsuite: Require lto in three tests

2024-10-06 Thread John David Anglin
Committed as obvious. Dave --- testsuite: Require lto in three tests 2024-10-06 John David Anglin gcc/testsuite/ChangeLog: * g++.dg/cpp0x/noexcept87.C: Require lto. * g++.dg/ext/pragma-unroll-lambda-lto.C: Likewise. * gcc.dg/enum-alias-3.c: Likewise. diff --git a/gc