Re: [PATCH v22 0/3] c: Add _Countof and

2025-05-16 Thread Joseph Myers
On Fri, 16 May 2025, Alejandro Colomar wrote: > Hmmm, I've been trying to find a compromise between readability and > simplicity, and I think I have something. I've seen some tests that > define assert() themselves. I like assert(3) because it's more > readable compared to a conditional plus abo

Re: [PATCH] RISC-V: Since the loop increment i++ is unreachable, the loop body will never execute more than once

2025-05-16 Thread Jin Ma
On Fri, 16 May 2025 09:18:45 -0600, Jeff Law wrote: > > > On 5/16/25 1:32 AM, Jin Ma wrote: > > Reported-by: huangcunjian > > > > gcc/ChangeLog: > > > > * config/riscv/riscv.cc (riscv_gpr_save_operation_p): Remove > > break and fixbug for elt index. > Ideally we'd have a testcase for w

Re: [PATCH v2] libstdc++: Preserve the argument type in basic_format_args [PR119246]

2025-05-16 Thread Tomasz Kaminski
On Thu, May 15, 2025 at 9:59 PM Tomasz Kaminski wrote: > > > On Thu, May 15, 2025 at 9:45 PM Rainer Orth > wrote: > >> Hi Tomasz, >> >> >> I've a local patch in tree to support __float128 on SPARC, so I'll try >> >> with an unmodified tree first. However, 2 days ago I could bootstrap >> >> with

Re: [PATCH 1/1] middle-end: Fix operation_could_trap_p for FIX_TRUNC_EXPR

2025-05-16 Thread Richard Biener
On Wed, May 14, 2025 at 4:06 PM Spencer Abson wrote: > > Floating-point to integer conversions can be inexact or invalid (e.g., due to > overflow or NaN). However, since users of operation_could_trap_p infer the > bool FP_OPERATION argument from the expression's type, FIX_TRUNC_EXPR is > consider

Re: [PATCH 3/3] cfgcleanup: small performance improvement in some cases

2025-05-16 Thread Richard Biener
On Tue, May 13, 2025 at 12:47 AM Andrew Pinski wrote: > > With some functions, there might be the case where every stack variable is a > live at the end of a > basic block. If that is the case then it is known that all stack variables > will conflict with each > other so there is no reason to go

Re: [PATCH v2] libstdc++: Preserve the argument type in basic_format_args [PR119246]

2025-05-16 Thread Rainer Orth
Hi Tomasz, > I am not aware of architectures where the additional specialization for > __float128 > would be useful, so if my patch of changing _GLIBCXX_FORMAT_F128 > 1 to > _GLIBCXX_FORMAT_F128 == 2 > addressed your build, I can submit it. with this patch, the sparc-sun-solaris2.11 bootstrap co

Re: [PATCH v1 00/10] RISC-V: Combine vec_duplicate + vsub.vv to vsub.vx on GR2VR cost

2025-05-16 Thread Robin Dapp
Excuse the delay, I was attending the RISC-V Summit Europe. The series LGTM. -- Regards Robin

[PATCH] RISC-V: Since the loop increment i++ is unreachable, the loop body will never execute more than once

2025-05-16 Thread Jin Ma
Reported-by: huangcunjian gcc/ChangeLog: * config/riscv/riscv.cc (riscv_gpr_save_operation_p): Remove break and fixbug for elt index. --- gcc/config/riscv/riscv.cc | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/ris

RE: [PATCH v1 00/10] RISC-V: Combine vec_duplicate + vsub.vv to vsub.vx on GR2VR cost

2025-05-16 Thread Li, Pan2
> Excuse the delay, I was attending the RISC-V Summit Europe. Thanks Robin, and never mind. Pan -Original Message- From: Robin Dapp Sent: Friday, May 16, 2025 3:26 PM To: Li, Pan2 ; gcc-patches@gcc.gnu.org Cc: juzhe.zh...@rivai.ai; kito.ch...@gmail.com; jeffreya...@gmail.com; rdapp...

[PATCH v1] RISC-V: Avoid scalar unsigned SAT_ADD test data duplication

2025-05-16 Thread pan2 . li
From: Pan Li Some of the previous scalar unsigned SAT_ADD test data are duplicated in different test files. This patch would like to move them into a shared header file, to avoid the test data duplication. The below test suites are passed for this patch series. * The rv64gcv fully regression te

[PATCH] Enable mcf thread model for aarch64-*-mingw*.

2025-05-16 Thread LIU Hao
This is a leftover of d6d7afcdbc04adb0ec42a44b2d7e05600945af42. After this change, configuration files of all three thread models are in 'libgcc/config/mingw/'. The patch has been bootstrapped on {x86_64,i686}-w64-mingw32. ARM64 port is still working in progress and I will keep an eye on it for

[PATCH] [PR120276] regcprop: Replace partial_subreg_p by ordered_p && maybe_lt

2025-05-16 Thread Jennifer Schmitz
The ICE in PR120276 resulted from a comparison of VNx4QI and V8QI using partial_subreg_p in the function copy_value during the RTL pass regcprop, failing the assertion in inline bool partial_subreg_p (machine_mode outermode, machine_mode innermode) { /* Modes involved in a subreg must be ordered

<    1   2