Re: [PATCH] Fortran: reject SAVE of a COMMON in a BLOCK construct [PR119199]

2025-03-10 Thread Thomas Koenig
Hi Harald, Regtested on x86_64-pc-linux-gnu.  OK for mainline? Looks good to me. Thanks for the patch! Best regards Thomas

Re: [RFC] [C]New syntax for the argument of counted_by attribute for C language

2025-03-10 Thread Martin Uecker
Am Montag, dem 10.03.2025 um 19:30 -0400 schrieb John McCall: > On 10 Mar 2025, at 18:30, Martin Uecker wrote: > > Am Montag, dem 10.03.2025 um 16:45 -0400 schrieb John McCall: > > > > .. > > > > > > > > > > > > While the next example is also ok in C++. > > > > > > > > > > > > constexpr int n

Re: [RFC] [C]New syntax for the argument of counted_by attribute for C language

2025-03-10 Thread Henrik Olsson
> On Mar 10, 2025, at 11:04 PM, Martin Uecker wrote: > > Am Montag, dem 10.03.2025 um 19:30 -0400 schrieb John McCall: >> On 10 Mar 2025, at 18:30, Martin Uecker wrote: >>> Am Montag, dem 10.03.2025 um 16:45 -0400 schrieb John McCall: > > > .. > >>> > >>> While the next example

Re: [PATCH v2] i386: Verify that argument registers are spilled properly

2025-03-10 Thread Uros Bizjak
On Sun, Mar 9, 2025 at 11:34 PM H.J. Lu wrote: > > While working on a local x86 patch, which passed the GCC testsuite, I got > a compiler error: > > In function ‘paravirt_read_msr’, > inlined from ‘perf_ibs_handle_irq’ at arch/x86/events/amd/ibs.c:1055:2: > ./arch/x86/include/asm/paravirt_type

[PATCH v2] driver: Fix multilib_os_dir and multiarch_dir for those target use TARGET_COMPUTE_MULTILIB

2025-03-10 Thread Kito Cheng
This patch fixes the multilib_os_dir and multiarch_dir for those targets that use TARGET_COMPUTE_MULTILIB, since the TARGET_COMPUTE_MULTILIB hook only update/fix the multilib_dir but not the multilib_os_dir and multiarch_dir, so the multilib_os_dir and multiarch_dir are not set correctly for those

[PATCH] Fix a pasto in ao_compare::compare_ao_refs

2025-03-10 Thread Martin Jambor
Hi, when reading the function ao_compare::compare_ao_refs I came accross what I believe to ba a copy-and-paste error which this patch fixes. Bootstrapped, LTO-bootstrapped and tested on x86_64-linux. OK for master? Thanks, Martin gcc/ChangeLog: 2025-03-10 Martin Jambor * tree-ss

RE: [PATCH] combine: Add REG_DEAD notes to the last instruction after a split [PR118914]

2025-03-10 Thread Andrew Pinski (QUIC)
> -Original Message- > From: Andrew Pinski (QUIC) > Sent: Wednesday, February 19, 2025 8:14 PM > To: gcc-patches@gcc.gnu.org > Cc: Andrew Pinski (QUIC) > Subject: [PATCH] combine: Add REG_DEAD notes to the last > instruction after a split [PR118914] > > So gcc.target/aarch64/rev16_2.c st

Re: The COBOL front end, version 3, now in 14 easy pieces

2025-03-10 Thread Richard Biener
On Mon, Mar 10, 2025 at 10:40 PM David Malcolm wrote: > > On Mon, 2025-03-10 at 19:07 +0100, Richard Biener wrote: > > On Mon, Mar 10, 2025 at 5:34 PM Richard Biener > > wrote: > > > > > > On Sat, Mar 8, 2025 at 12:12 AM Robert Dubner > > > wrote: > > > > > > > > Richard and Jakub, and everybody

Re: [PATCH] c, v2: do not warn about truncating NUL char when initializing nonstring arrays [PR117178]

2025-03-10 Thread Kees Cook
On Mon, Mar 10, 2025 at 01:03:44PM -0700, Kees Cook wrote: > Hi! Thanks again for continuing to poke at this. Even with commit > 1301e18f69ce ("gimple-ssa-warn-access: Adjust maybe_warn_nonstring_arg > for nonstring multidimensional arrays [PR117178]"), this is still not > working. Does the above t

Re: [PATCH] LoongArch: Fix ICE when trying to recognize bitwise + alsl.w pair [PR119127]

2025-03-10 Thread Lulu Cheng
LGTM. Thanks. 在 2025/3/10 下午2:40, Xi Ruoyao 写道: When we call loongarch_reassoc_shift_bitwise for _alsl_reversesi_extend, the mask is in DImode but we are trying to operate it in SImode, causing an ICE. To fix the issue sign-extend the mask into the mode we want. And also specially handle the

Re: [PATCH] libgcc: Fix up unwind-dw2-btree.h [PR119151]

2025-03-10 Thread Thomas Neumann
Hi Jakub, What differs from the textbook implementations is mostly that the leaf nodes don't include just address as a key, but address range, address + size (where we don't insert any ranges with zero size) and the lookups can be performed for any address in the [address, address + size) range.

Re: [PATCH] gimple-ssa-warn-access: Adjust maybe_warn_nonstring_arg for nonstring multidimensional arrays [PR117178]

2025-03-10 Thread Richard Biener
> Am 10.03.2025 um 08:35 schrieb Jakub Jelinek : > > Hi! > > I've rushed up the PR117178 multi-dimensional nonstring support. Before > a week off I've just bootstrapped/regtested the actual change, so > handle_nonstring_attribute/get_attr_nonstring_decl adjustments after playing > with simpl

[PATCH] libgcc: Fix up unwind-dw2-btree.h [PR119151]

2025-03-10 Thread Jakub Jelinek
Hi! The following testcase shows a bug in unwind-dw2-btree.h. In short, the header provides lock-free btree data structure (so no parent link on nodes, both insertion and deletion are done in top-down walks with some locking of just a few nodes at a time so that lookups can notice concurrent modif

[PATCH] libgcc: Formatting fixes for unwind-dw2-btree.h

2025-03-10 Thread Jakub Jelinek
Hi! Studying unwind-dw2-btree.h was really hard for me because the formatting is wrong or weird in many ways all around the code and that kept distracting my attention. That includes all kinds of things, including wrong indentation, using {} around single statement substatements, excessive use of

Ping: [PATCH] LoongArch: combine related slli operations

2025-03-10 Thread Zhou Zhao
Ping 在 2025/1/7 下午8:45, Zhou Zhao 写道: 在 2025/1/7 下午7:49, Lulu Cheng 写道: 在 2025/1/2 下午5:46, Zhou Zhao 写道: If SImode reg is continuous left shifted twice, combine related instruction to one. gcc/ChangeLog: * config/loongarch/loongarch.md (extsv_ashlsi3):     New template Hi, zhaozhou:

Re: [Ping, Patch, www-docs, Fortran, Coarray-ABI] Announce coarray-ABI changes in gfortran-15

2025-03-10 Thread Andre Vehreschild
Hi Steve and Jerry, thanks for the review and the proposed changes. I have based on them, but needed to adapt some places, because the meaning was changed. Can you please take another look? Jerry, where do I find this check-script? In bin/ nothing jumps out at me to be a check-script. Thanks,

Re: AArch64: Turn off outline atomics with -mcmodel=large (PR112465)

2025-03-10 Thread Ramana Radhakrishnan
On Fri, Mar 7, 2025 at 11:07 PM Richard Sandiford wrote: > > Wilco Dijkstra writes: > > Hi Richard, > > > >>> Basically the small and large model are fundamentally incompatible. The > >>> infamous > >>> "dumb linker" approach means it doesn't try to sort sections, so an ADRP > >>> relocation >

Re: Ping: [PATCH] LoongArch: combine related slli operations

2025-03-10 Thread Lulu Cheng
在 2025/3/10 下午3:24, Zhou Zhao 写道: Ping Hi, You need to send another patch for version V2 with modifications. However, I am currently testing the value of align, and GCC15 is already in stage 4, so this patch might have to wait until the GCC16 stage 1 phase before it can be integrated.

Re: [PATCH] libgcc: Fix up unwind-dw2-btree.h [PR119151]

2025-03-10 Thread Richard Biener
 > Am 10.03.2025 um 08:16 schrieb Jakub Jelinek : > > Hi! > > The following testcase shows a bug in unwind-dw2-btree.h. > In short, the header provides lock-free btree data structure (so no parent > link on nodes, both insertion and deletion are done in top-down walks > with some locking of ju

Re: [PATCH] contrib: Clean up outdated parts of gcc-git-customization.sh

2025-03-10 Thread Richard Earnshaw (lists)
On 07/03/2025 21:12, Jonathan Wakely wrote: > It's very unlikely that anybody is still using the old remotes/$user Git > repo setup and still needs this script to be able to migrate it to the > remotes/users/$user structure. Simplify the script by removing those > parts. > > This fixes an error th

Re: [PATCH] arm: [MVE] Fix predicates for vec_cmp, vec_vcmpu and vcond_mask (PR 115439)

2025-03-10 Thread Richard Earnshaw (lists)
On 16/01/2025 14:20, Christophe Lyon wrote: > When compiling c-c++-common/vector-compare-3.c with > -march=armv8.1-m.main+mve+fp.dp -mfloat-abi=hard -mfpu=auto > (which enables MVE), we fail to match vcond_mask because operand 3 has > s_register_operand as predicate for a MVE_VPRED mode, but we try

[PATCH] s390: fix delegitimization of addresses

2025-03-10 Thread Juergen Christ
In legitimize_pic_address we create a (const (unspec ... UNSPEC_GOTENT)) in the GOT offset might be >= 4k. However, the s390_delegitimize_address does not contain a case for this scenario. gcc/ChangeLog: * config/s390/s390.cc (s390_delegitimize_address): Add missing case. gcc/testsuite/

Re: c++/modules: Stream section, tls_model, and comdat_group

2025-03-10 Thread Jason Merrill
On 3/10/25 9:52 AM, Nathaniel Shead wrote: Bootstrapped and regtested on x86_64-pc-linux-gnu, OK for trunk? Or should this wait for GCC16? -- >8 -- While looking at PR c++/119154 I noticed some more properties that we don't stream or check properly. This patch adds the ones we were missing, an

[pushed][PR114991][IRA]: Improve reg equiv invariant calculation

2025-03-10 Thread Vladimir Makarov
The following patch solves performance issue mentioned in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114991 The patch was successfully bootstrapped and tested on x86_64, aarch64, ppc64le. commit e355fe414aa3aaf215c7dd9dd789ce217a1b458c Author: Vladimir N. Makarov Date: Mon Mar 10 16:26:5

Re: The COBOL front end, version 3, now in 14 easy pieces

2025-03-10 Thread James K. Lowden
On Mon, 10 Mar 2025 18:05:21 +0100 Jakub Jelinek wrote: > Designated initializers are C++20, so you should just avoid that. So, > I'd recommend just: > cbl_field_data_t data = { /* memsize= */capacity_cast(len), > /* capacity= */capacity_cast(len), >

New German PO file for 'cpplib' (version 15-b20250216)

2025-03-10 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'cpplib' has been submitted by the German team of translators. The file is available at: https://translationproject.org/latest/cpplib/de.po (This file, 'cpplib-15-b20250216.

Contents of PO file 'cpplib-15-b20250216.de.po'

2025-03-10 Thread Translation Project Robot
cpplib-15-b20250216.de.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

Re: [patch, fortran] Fix PR 119078, putting a procedure in an abstract interface into global namespace

2025-03-10 Thread Harald Anlauf
Hi Thomas, Am 10.03.25 um 21:01 schrieb Thomas Koenig: Hello world, the attached patch makes sure that procedures from abstract interfaces and dummy arguments are not put into the global symbol table, and are not checked against global symbols. Regression-tested. OK for trunk? Best regards

Re: [RFC] [C]New syntax for the argument of counted_by attribute for C language

2025-03-10 Thread John McCall
On 10 Mar 2025, at 15:34, Martin Uecker wrote: > Am Montag, dem 10.03.2025 um 15:00 -0400 schrieb John McCall: >> That said, my preference is still to just give preference to the field name, >> which sidesteps any need for disambiguation syntax and avoids this whole >> problem where structs can be

Re: [COMMITTED] Sanitizer: Mention -g option in documentation [PR56682]

2025-03-10 Thread Christophe Lyon
Hi! On Fri, 7 Mar 2025 at 23:57, Sandra Loosemore wrote: > > gcc/ChangeLog > PR sanitizer/56682 > * doc/invoke.texi (Instrumentation Options): Document that -g > is useful with -fsanitize=thread and -fsanitize=address. > Also mention -fno-omit-frame-pointer per th

[committed] libphobos: Default to libc closefrom in spawnProcessPosix [PR119112]

2025-03-10 Thread Iain Buclaw
Hi, This is a backport of two changes in upstream Phobos. - The current implementation of spawnProcessPosix is broken on systems with a large `ulimit -n' because it always OOMs making it impossible to spawn processes. Using the libc implementation, when available, for doing file descriptor operat