RE: [PATCH] cobol: Implement CXXFLAGS_FOR_COBOL.

2025-07-13 Thread Robert Dubner
Thank you. I didn't respond sooner because I was away from the computer for a few days. And thank you for giving me some clarity on that line I labeled, "I don't understand..." > -Original Message- > From: Iain Buclaw > Sent: Thursday, July 10, 2025 15:11 > To: Robert Dubner ; Andreas

[wwwdocs] Remove contribute.html text about including ChangeLog in the patch

2025-07-13 Thread Jonathan Wakely
--- This is an anachronism: ChangeLog entries should never be part of the patch, because nobody should be editing them locally and including them in a commit. Whether you can self-approve or not isn't relevant. OK for wwwdocs? htdocs/contribute.html | 5 ++--- 1 file changed, 2 insertions(+), 3

[PATCH] Factor out thread model detection with `GCC_AC_THREAD_MODEL`

2025-07-13 Thread John Ericson
From: John Ericson This macro deduplicates the $CC -v 2>&1 | sed -n 's/^Thread model: //p' check that was occurring in various runtime libs. Additionally, as a bit of an Easter egg, this also allows overriding what the compiler would return by setting the `gcc_cv_target_thread_file` cache

Re: [wwwdocs] Remove contribute.html text about including ChangeLog in the patch

2025-07-13 Thread Jakub Jelinek
On Sun, Jul 13, 2025 at 05:28:21PM +0100, Jonathan Wakely wrote: > --- > > This is an anachronism: ChangeLog entries should never be part of the > patch, because nobody should be editing them locally and including them > in a commit. Whether you can self-approve or not isn't relevant. > > OK for

[PUSHED] tree: Add include to tm_p.h to tree.cc [PR120866]

2025-07-13 Thread Andrew Pinski
After r16-1738-g0337e3c2743ca0, a call to ASM_GENERATE_INTERNAL_LABEL was done without including tm_p.h. This does not break most targets as ASM_GENERATE_INTERNAL_LABEL macro function does not call target specific functions from it; mostly just sprintf. It does however break pdp11-aout and powerpc-

[RFC 0/1] fortran: TARGET_CLONES attribute argument parsing syntax

2025-07-13 Thread ZAMBAR
From: ZAMBAR Hi all, I would like to seek feedback on the implementation approach for the TARGET_CLONES attribute in gfortran, specifically regarding the syntax for parsing multiple target specifications. Currently, I have implemented TARGET_CLONES support for Fortran using a C-style comma-s

Re: [PATCH] middle-end: Fix typo in gimple.h

2025-07-13 Thread Jonathan Wakely
On 13/07/25 07:50 +, Benjamin Wu wrote: 2025-07-13 Ben Wu * gimple.h (GTMA_DOES_GO_IRREVOCABLE): Fix typo. Tested on x86_64-pc-linux-gnu. Will someone be able to commit this typo fix for me? The patch doesn't apply, because of the ChangeLog part. Please do *not* include changes to

Re: [PATCH] Allow explicitly specifying the thread model for runtime libs

2025-07-13 Thread John Ericson
I reached out on IRC, and got some feedback on this patch. In particular, Arsen thought the idea made some sense, but that while there was as-yet no long-term policy or design decision on this sort of thing, a proper --with-* flag would wrongly appear to instantly-stable, but a mere autoconf cac

[RFC 1/1] fortran: Add TARGET_CLONES attribute support

2025-07-13 Thread ZAMBAR
From: ZAMBAR This patch implements the TARGET_CLONES attribute for Fortran functions, using C-style comma-separated syntax for target specifications. The implementation adds: - Parsing support for TARGET_CLONES("target1", "target2", ...) - Integration with existing multiple target infrastructure

[committed] cobol: Minor changes to genapi.cc to eliminate CPPCHECK warnings.

2025-07-13 Thread Robert Dubner
Subject: [PATCH] cobol: Minor changes to genapi.cc to eliminate CPPCHECK warnings. Several hundred cppcheck warnings were eliminated. Most of these changes were replacing C-style casts, checking for NULL pointers, establishing some variables and formal parameters as const, and moving some variab

[PATCH v3] x86: Update MMX moves to support all 1s vectors

2025-07-13 Thread H.J. Lu
On Sat, Jul 12, 2025 at 7:51 PM Uros Bizjak wrote: > > On Sat, Jul 12, 2025 at 1:41 PM H.J. Lu wrote: > > > > On Sat, Jul 12, 2025 at 5:58 PM Uros Bizjak wrote: > > > > > > On Sat, Jul 12, 2025 at 11:52 AM H.J. Lu wrote: > > > > > > > > On Sat, Jul 12, 2025 at 5:03 PM Uros Bizjak wrote: > > >

Re: [PATCH v3] x86: Update MMX moves to support all 1s vectors

2025-07-13 Thread Uros Bizjak
On Mon, Jul 14, 2025 at 2:14 AM H.J. Lu wrote: > > On Sat, Jul 12, 2025 at 7:51 PM Uros Bizjak wrote: > > > > On Sat, Jul 12, 2025 at 1:41 PM H.J. Lu wrote: > > > > > > On Sat, Jul 12, 2025 at 5:58 PM Uros Bizjak wrote: > > > > > > > > On Sat, Jul 12, 2025 at 11:52 AM H.J. Lu wrote: > > > > >

Re: [PATCH] MicroBlaze : Enhance support for atomics. Fix PR118280

2025-07-13 Thread Gopi Kumar Bulusu
On Fri, Jul 11, 2025 at 8:12 PM Michael Eager wrote: > On 7/10/25 4:41 AM, Gopi Kumar Bulusu wrote: > > > > namaskaaram > > Hi Gopi! > > > > > Please find the patch attached. This addresses regression for MicroBlaze > > (PR118280) > > Neal Frager posted a different patch (or an RFC) to address pr

Re: [RFC v2] c++: Quoting in -fmodules-mapper [PR110153]

2025-07-13 Thread Nicolas Werner
Am Freitag, 11. Juli 2025, 18:42:34 Mitteleuropäische Sommerzeit schrieb Jason Merrill: > On 7/10/25 4:41 PM, Nicolas Werner wrote: > > Users might be using a space in their build directory path. To allow > > specifying such a root for the module mapper started by GCC, we need the > > command to a

[PATCH] middle-end: Fix typo in gimple.h

2025-07-13 Thread Benjamin Wu
2025-07-13 Ben Wu * gimple.h (GTMA_DOES_GO_IRREVOCABLE): Fix typo. Tested on x86_64-pc-linux-gnu. Will someone be able to commit this typo fix for me? Thanks! --- 7 gcc/ChangeLog | 4 8 gcc/gimple.h | 2 +- 9 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/gcc

[PATCH] fortran: Factor array descriptor references

2025-07-13 Thread Mikael Morin
Regression tested on x86_64-pc-linux-gnu. OK for master? -- >8 -- Save subexpressions of array descriptor references to variables, so that all the expressions using the descriptor as base object benefit from a simplified reference using the variables. This limits the size of the expressions gen

[PATCH] i386: Remove KEYLOCKER related feature since Panther Lake and Clearwater Forest

2025-07-13 Thread Haochen Jiang
Hi all, According to July 2025 SDM, Key locker will no longer be supported on hardware 2025 onwards. This means for Panther Lake and Clearwater Forest, the feature will not be enabled. Remove them from those two platforms. Ok for trunk and backport to GCC14/15? Thx, Haochen gcc/ChangeLog:

Re: [PATCH] libstdc++: Do not use list-initialization in std::span members [PR120997]

2025-07-13 Thread Tomasz Kaminski
On Tue, Jul 8, 2025 at 11:46 PM Jonathan Wakely wrote: > As the bug report shows, for span the return statements of > the form `return {data(), count};` will use the new C++26 constructor, > span(initializer_list). > > Although the conversions from data() to bool and count to bool are > narrowing

[r16-2197 Regression] FAIL: g++.dg/torture/pr58552.C -Os (test for excess errors) on Linux/x86_64

2025-07-13 Thread haochen.jiang
On Linux/x86_64, 385d9937f0e23cbf9c62f0b2553a33ff70e56ecf is the first bad commit commit 385d9937f0e23cbf9c62f0b2553a33ff70e56ecf Author: Jan Hubicka Date: Fri Jul 11 13:01:13 2025 +0200 Rewrite assign_discriminators caused FAIL: c-c++-common/torture/pr116156-1.c -O1 (test for excess

RE: [PATCH] i386: Remove KEYLOCKER related feature since Panther Lake and Clearwater Forest

2025-07-13 Thread Liu, Hongtao
> -Original Message- > From: Jiang, Haochen > Sent: Monday, July 14, 2025 10:59 AM > To: gcc-patches@gcc.gnu.org > Cc: Liu, Hongtao ; ubiz...@gmail.com > Subject: [PATCH] i386: Remove KEYLOCKER related feature since Panther Lake > and Clearwater Forest > > Hi all, > > According to Jul

Re: [PATCH] aarch64: Implement sme2+faminmax extension.

2025-07-13 Thread Alfie Richards
On 09/07/2025 09:28, Kyrylo Tkachov wrote: Hi Alfie, On 7 Jul 2025, at 10:46, Alfie Richards wrote: Hello all, This patch implements the couple of amin/amax instructions that are part of SME2 + faminmax. Regression testsed and bootstrapped for Aarch64. Thanks, Alfie -- >8 -- Implements t

Re: [PATCH v4 0/6] Implement mdspan.

2025-07-13 Thread Tomasz Kaminski
Hi Luc, While running the libc++ test on libstdc++ we have found the following issue in our implementation. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=121061 Would you be interested in looking into fixing this? Also, libc++ makes the default constructor of mdpsan conditionally noexcept as exten