Re: [PATCH] arc: Add const attribute support for mathematical ARC builtins

2025-10-07 Thread Kees Cook
On Tue, Oct 07, 2025 at 09:02:21PM +, Qing Zhao wrote: > Hi, Kees, > > I took a look at the patch today. > Overall, I think the patch is good and should work well. Thanks! > My major questions are: > > 1. Are the five functions in the current list the only functions in > __builtin_arc_**

Re: [PATCH] arc: Add const attribute support for mathematical ARC builtins

2025-10-07 Thread Qing Zhao
Hi, Kees, I took a look at the patch today. Overall, I think the patch is good and should work well. My major questions are: 1. Are the five functions in the current list the only functions in __builtin_arc_*** that have const attribute? 2. In addition to “const” attribute, are there any o

[PATCH v3 5/5] xtensa: constantsynth: Update to version 2

2025-10-07 Thread Takayuki 'January June' Suwa
This patch completely replaces the existing "constantsynth" with a new implementation, which has become unsightly due to the extension. This new version offers the following benefits: - Independence from the insn splitting mechanism. No define_split descriptions are required - Resource sav

[PATCH v4 0/2] Multilib changes

2025-10-07 Thread Aleksandar Rakic
Hi Joseph, Thank you for your detailed feedback. The updated patches have been restructured and revised to address your comments: - Patch separation has been clarified: All target-independent logic is now contained in the first patch, while the second patch includes only MIPS-specific change

Re: PING: [PATCH] libbid: Set rounding mode to round-to-nearest for _Decimal128 arithmetic

2025-10-07 Thread Akkas, Ahmet
Your updates look good to me. We can go ahead and merge these changes. Thank you very much for your help! Best regards, Ahmet From: H.J. Lu Sent: Monday, October 6, 2025 6:12 PM To: Akkas, Ahmet Cc: Cornea, Marius ; Hongtao Liu ; GCC Patches ; Liu, Hongtao ;

Re: [PATCH] Fortran: fix warnings for symbols with C binding and declared PRIVATE [PR49111]

2025-10-07 Thread Jerry D
On 10/7/25 1:13 PM, Harald Anlauf wrote: Dear All, the attached patch addresses a rather old (> 14 years) issue. We generated warnings for standard conforming code, where a symbol was given a bind(c) attribute and at the same time declared PRIVATE. I checked a bunch of compilers, and none gave

Re: [PATCH] dwarf: Save bit stride information array type entry [PR121964]

2025-10-07 Thread Jakub Jelinek
On Tue, Oct 07, 2025 at 06:55:25PM +, Tamar Christina wrote: > Consider: > > #include > > svbool_t f8 () > { > return svptrue_b8 (); > } > > svbool_t f16 () > { > return svptrue_b16 (); > } > > The first one has every bit significant, and the second one it's every 2nd > bit. > But wit

Re: [PATCH] libstdc++: Remove unused config/cpu/i486/atomicity.h file [PR122172]

2025-10-07 Thread Jonathan Wakely
On Tue, 7 Oct 2025 at 19:53, Jonathan Wakely wrote: > > On Tue, 7 Oct 2025 at 18:31, Hans-Peter Nilsson wrote: > > > > > From: Jonathan Wakely > > > Date: Tue, 7 Oct 2025 17:32:55 +0100 > > > > > On Tue, 7 Oct 2025 at 17:29, Hans-Peter Nilsson wrote: > > > > > > > > > From: Jonathan Wakely > >

Re: [PATCH] libstdc++: Remove unused config/cpu/i486/atomicity.h file [PR122172]

2025-10-07 Thread Jonathan Wakely
On Tue, 7 Oct 2025 at 18:31, Hans-Peter Nilsson wrote: > > > From: Jonathan Wakely > > Date: Tue, 7 Oct 2025 17:32:55 +0100 > > > On Tue, 7 Oct 2025 at 17:29, Hans-Peter Nilsson wrote: > > > > > > > From: Jonathan Wakely > > > > > > > This file is no longer used, because we always use the defin

Re: [Patch, fortran] PRs 93175, 102240 and 102686 -Fix PDT parameter substitution problems

2025-10-07 Thread Harald Anlauf
Am 07.10.25 um 18:56 schrieb Jerry D: On 10/7/25 2:19 AM, Paul Richard Thomas wrote: Hi All, The real content of this patch is far simpler that the size would seem to imply. I started replacing 8 spaces with tabs in primary.cc and resolve.cc. Once started, I couldn't stop :-) This job will be c

Re: [Patch, fortran] PRs 93175, 102240 and 102686 -Fix PDT parameter substitution problems

2025-10-07 Thread Jerry D
On 10/7/25 2:19 AM, Paul Richard Thomas wrote: Hi All, The real content of this patch is far simpler that the size would seem to imply. I started replacing 8 spaces with tabs in primary.cc and resolve.cc. Once started, I couldn't stop :-) This job will be completed at another time. The changes

[committed] libstdc++: Fix -Wparentheses warning in std::mul_sat

2025-10-07 Thread Jonathan Wakely
libstdc++-v3/ChangeLog: * include/bits/sat_arith.h (mul_sat): Add parentheses around operands. --- Tested powerpc64le-linux. Pushed to trunk. libstdc++-v3/include/bits/sat_arith.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libstdc++-v3/include/bits/sat_a

Re: [PATCH v5 3/3] fmv: Add -ftarget-clones-table option support

2025-10-07 Thread Yangyu Chen
> On 5 Oct 2025, at 05:15, Jeff Law wrote: > > > On 10/3/25 12:27 PM, Yangyu Chen wrote: >> This patch adds support for target_clones table option. The >> target_clones table option allows users to specify multiple versions >> of a function and select the version at runtime based on the speci

Re: Improve profile update in merge_blocks

2025-10-07 Thread Richard Biener
On Tue, Oct 7, 2025 at 12:39 PM Jan Hubicka wrote: > > > > +/* If basic block is empty, we can remove conditionals that controls > > > + its execution. However in some cases the empty BB can stay live > > > + (such as when it was a header of empty loop). In this case we > > > + need to upd

[Patch, fortran] PRs 93175, 102240 and 102686 -Fix PDT parameter substitution problems

2025-10-07 Thread Paul Richard Thomas
Hi All, The real content of this patch is far simpler that the size would seem to imply. I started replacing 8 spaces with tabs in primary.cc and resolve.cc. Once started, I couldn't stop :-) This job will be completed at another time. The changes listed in the ChangeLog entries for array.cc, dec

Re: [PATCH] c++: Fix ICE with concept used as function [PR116477]

2025-10-07 Thread Egas Carvalho Perdigão Gonçalves Ribeiro
On 2025-10-06 20:38, Patrick Palka wrote: Some context: GCC represents templated and non-templated expressions differently, and the processing_template_decl flag controls whether our assorted semantic analysis and AST building functions should produce a templated vs non-templated tree. And we

Re: [PATCH] doc: mention -Wuninitialized vs CCP

2025-10-07 Thread Richard Biener
On Tue, Oct 7, 2025 at 4:22 AM Sam James wrote: > > Sandra Loosemore writes: > > > On 10/6/25 19:03, Andrew Pinski wrote: > >> On Mon, Oct 6, 2025 at 5:58 PM Sam James wrote: > >>> > >>> CCP interacts poorly with -Wuninitialized in some cases by assuming a > >>> value > >>> which stops us warni

[PATCH] RISC-V: Detect wrap in shuffle_series_pattern [PR121845].

2025-10-07 Thread Robin Dapp
Hi, In shuffle_series_pattern we use series_p to determine if the permute mask is a simple series. This didn't take into account that series_p also returns true for e.g. {0, 3, 2, 1} where the step is 3 and the indices form a series modulo 4. We emit vid + vmul in order to synthesize a series.