Re: [PATCH htdocs] gcc-14: document -ftrampoline-impl

2025-02-09 Thread Iain Sandoe
> On 10 Feb 2025, at 08:28, Sam James wrote: > > --- > Iain, does this look OK? > > htdocs/gcc-14/changes.html | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html > index ba4780ca..2db442b2 100644 > --- a/htdocs/gcc-14/change

Re: [PATCH 0/3] GCC13/GCC12 backport [PR108707][PR109610]

2025-02-09 Thread Richard Biener
On Mon, Feb 10, 2025 at 6:46 AM liuhongt wrote: > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108707#c9 > > >Pranav Gorantla 2025-02-06 04:30:05 UTC > >Facing similar issue in gcc-13. Is it possible to backport the fix of this > >Bug 108707 and Bug 109610 to gcc-13, gcc-12 as well. > > This se

[PATCH htdocs] gcc-14: document -ftrampoline-impl

2025-02-09 Thread Sam James
--- Iain, does this look OK? htdocs/gcc-14/changes.html | 7 +++ 1 file changed, 7 insertions(+) diff --git a/htdocs/gcc-14/changes.html b/htdocs/gcc-14/changes.html index ba4780ca..2db442b2 100644 --- a/htdocs/gcc-14/changes.html +++ b/htdocs/gcc-14/changes.html @@ -123,6 +123,13 @@ You may

RE: [PATCH] i386: Fix AVX512BW intrin header with __OPTIMIZE__ [PR 118813]

2025-02-09 Thread Liu, Hongtao
> -Original Message- > From: Jiang, Haochen > Sent: Monday, February 10, 2025 2:10 PM > To: gcc-patches@gcc.gnu.org > Cc: Liu, Hongtao ; ubiz...@gmail.com > Subject: [PATCH] i386: Fix AVX512BW intrin header with __OPTIMIZE__ [PR > 118813] > > Hi all, > > When moving intrins around for

[PATCH] i386: Fix AVX512BW intrin header with __OPTIMIZE__ [PR 118813]

2025-02-09 Thread Haochen Jiang
Hi all, When moving intrins around for AVX10 implementation in GCC 14, the intrin _kshiftli_mask32 and _kshiftri_mask32 are wrongly wrapped by "#if __OPTIMIZE__" instead of "#ifdef __OPTIMIZE__", leading to the intrin file not `-Wsystem-headers -Wundef` clean since r14-4490. Ok for trunk? Thx, H

[PATCH 3/3] Adjust testcases after better RA decision.

2025-02-09 Thread liuhongt
After optimization for RA, memory op is not propagated into instructions(>1), and it make testcases not generate vxorps since the memory is loaded into the dest, and the dest is never unused now. So rewrite testcases to make the codegen more stable. gcc/testsuite/ChangeLog: * gcc.target/

[PATCH 1/3] Use NO_REGS in cost calculation when the preferred register class are not known yet.

2025-02-09 Thread liuhongt
gcc/ChangeLog: PR rtl-optimization/108707 * ira-costs.cc (scan_one_insn): Use NO_REGS instead of GENERAL_REGS when preferred reg_class is not known. gcc/testsuite/ChangeLog: * gcc.target/i386/pr108707.c: New test. (cherry picked from commit 0368d169492017cfab5622

Re: [PATCH 0/3] GCC13/GCC12 backport [PR108707][PR109610]

2025-02-09 Thread Hongtao Liu
On Mon, Feb 10, 2025 at 1:43 PM liuhongt wrote: > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108707#c9 > > >Pranav Gorantla 2025-02-06 04:30:05 UTC > >Facing similar issue in gcc-13. Is it possible to backport the fix of this > >Bug 108707 and Bug 109610 to gcc-13, gcc-12 as well. > > This se

[PATCH 2/3] Only use NO_REGS in cost calculation when !hard_regno_mode_ok for GENERAL_REGS and mode.

2025-02-09 Thread liuhongt
r14-172-g0368d169492017 replaces GENERAL_REGS with NO_REGS in cost calculation when the preferred register class are not known yet. It regressed powerpc PR109610 and PR109858, it looks too aggressive to use NO_REGS when mode can be allocated with GENERAL_REGS. The patch takes a step back, still use

[PATCH 0/3] GCC13/GCC12 backport [PR108707][PR109610]

2025-02-09 Thread liuhongt
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=108707#c9 >Pranav Gorantla 2025-02-06 04:30:05 UTC >Facing similar issue in gcc-13. Is it possible to backport the fix of this Bug >108707 and Bug 109610 to gcc-13, gcc-12 as well. This series is to ask approval for the backport of r14-172 and r14-125

Re: [PATCH] i386: Change RTL representation of bt[lq] [PR118623]

2025-02-09 Thread Uros Bizjak
On Sun, Feb 9, 2025 at 11:31 PM Jakub Jelinek wrote: > > On Sun, Feb 09, 2025 at 09:24:30AM +0100, Uros Bizjak wrote: > > "For commutative and comparison operators, a constant is always made > > the second operand." > > Isn't that just for commutative comparison operators (eq, ne, ordered, > unord

Re: [PATCH] i386: Change RTL representation of bt[lq] [PR118623]

2025-02-09 Thread Jakub Jelinek
On Sun, Feb 09, 2025 at 09:24:30AM +0100, Uros Bizjak wrote: > "For commutative and comparison operators, a constant is always made > the second operand." Isn't that just for commutative comparison operators (eq, ne, ordered, unordered, ltgt, uneq)? Compare itself even isn't RTX_COMPARE at all, i

Re: [PATCH 0/1] gdc: define ELFv1, ELFv2 and D_PPCUseIEEE128 versions for powerpc

2025-02-09 Thread Iain Buclaw
Excerpts from liushuyu's message of Februar 6, 2025 2:02 am: > From: Zixing Liu > > This set of patches will add proper IEEE 128 quad precision marking to GDC > compiler, so that it works with the new changes in D standard library > where POWER system can use any math functions inside the standar

[committed][RISC-V][PR target/115123] Fix testsuite fallout from sinking heuristic change

2025-02-09 Thread Jeff Law
Code sinking is just semantic preserving code motions, so it's a lot like scheduling in that code motions can change the vector configuration needed at various program points. That in turn can also change the number of vsetvls as we may or may not be able to merge them after the code motions.

Re: [committed][rtl-optimization/116244] Don't create bogus regs in alter_subreg

2025-02-09 Thread Jeff Law
On 2/9/25 1:10 AM, Georg-Johann Lay wrote: The .ira dump has several paradoxical subregs like: (insn 22 21 60 4 (set (reg/v:SI 51 [ val32 ]) (subreg:SI (reg:HI 53 [ t$val ]) 0)) "pr116389-red.c":14:14 146 {*movsi_split} (insn 27 26 28 5 (set (reg/v:SI 51 [ val32 ]) (subreg

[committed][PR middle-end/117263] Avoid unused-but-set warning in genautomata

2025-02-09 Thread Jeff Law
This is a trivial bug where a user wanted to define NDEBUG when building genautomata, presumably trying to debug its behavior. This resulted in a unused-but-set warning which caused the build to fail. Dario included the trivial fixes in the PR which I put through the usual bootstrap & regress

Re: [PATCH v2] RISC-V: Vector pesudoinsns with x0 operand to use imm 0

2025-02-09 Thread Vineet Gupta
On 2/8/25 23:02, Jeff Law wrote: > On 2/7/25 9:34 PM, Vineet Gupta wrote: >> A couple of Vector pseudoinstructions use x0 scalar which being regfile >> crosser could be inefficient on certain wider uarches. >> >> Use the imm 0 form, which should be functionally equivalent. >> >> pseudoinsn

Re: [PATCH] c++/modules: Better handle no-linkage decls in unnamed namespaces [PR118799]

2025-02-09 Thread Nathaniel Shead
On Sun, Feb 09, 2025 at 01:16:00AM +1100, Nathaniel Shead wrote: > Tested on x86_64-pc-linux-gnu, OK for trunk if full bootstrap + regtest > passes? > > -- >8 -- > > There are two issues with no-linkage decls (e.g. explicit type aliases) > in unnamed namespaces that this patch fixes. > > Firstly

[patch,avr] Add -mno-call-main to tweak running main()

2025-02-09 Thread Georg-Johann Lay
On devices with very limited resources, it may be desirable to run main in a more efficient way than provided by the startup code XCALL main XJMP exit from section .init9. In AVR-LibC v2.3, that code has been moved to libmcu.a, hence symbol __call_main can be satisfied so that the respec

Re: [Patch, Fortran] Fix PR 24878, checking actual arguments against global symbols

2025-02-09 Thread Thomas Koenig
Am 08.02.25 um 22:46 schrieb Harald Anlauf: looks good, just two minor comments: +  actual_name = act_sym->name ? act_sym->name : act_sym->name; Why not just +  actual_name = act_sym->name; ? That was a leftover from a previous commit. +  gfc_error ("Type mismatch pa

Re: [PATCH] i386: Change RTL representation of bt[lq] [PR118623]

2025-02-09 Thread Uros Bizjak
On Sat, Feb 8, 2025 at 9:40 AM Jakub Jelinek wrote: > > Hi! > > The following testcase is miscompiled because of RTL represententation > of bt{l,q} insn followed by e.g. j{c,nc} being misleading to what it > actually does. > Let's look e.g. at > (define_insn_and_split "*jcc_bt" > [(set (pc) >

Re: [committed][rtl-optimization/116244] Don't create bogus regs in alter_subreg

2025-02-09 Thread Georg-Johann Lay
CCing Denis Am 08.02.25 um 23:51 schrieb Jeff Law: On 2/8/25 1:52 PM, Georg-Johann Lay wrote: Am 08.02.25 um 18:23 schrieb Jeff Law: On 2/8/25 3:04 AM, Georg-Johann Lay wrote: That test case from https://gcc.gnu.org/bugzilla/show_bug.cgi? id=116389#c7 still ICEs with that change in http://gc