Re: [PATCH] Add std::deque shrink_to_fit test

2025-04-13 Thread Tomasz Kaminski
On Sun, Apr 13, 2025 at 12:13 PM François Dumont wrote: > > On 11/04/2025 08:36, Tomasz Kaminski wrote: > > > > On Thu, Apr 10, 2025 at 10:47 PM Jonathan Wakely > wrote: > >> On 10/04/25 22:36 +0200, François Dumont wrote: >> >After running the test with -fno-exceptions option we rather need thi

Re: [PATCH] libstdc++: Fix std::string construction from volatile char* [PR119748]

2025-04-13 Thread Tomasz Kaminski
On Sat, Apr 12, 2025 at 4:01 PM Jonathan Wakely wrote: > My recent r15-9381-g648d5c26e25497 change assumes that a contiguous > iterator with the correct value_type can be converted to a const charT* > but that's not true for volatile charT*. The optimization should only be > done if it can be con

Re: [Fortran, Patch, Teams, 0/5] Improve on Fortran 2018 teams support

2025-04-13 Thread Andre Vehreschild
Hi Jerry, thank you very much for the review. I would love to fix the nits you found, but I don't see, what you see. Can you elaborate? May be some mail client has removed something, or I am missing something. Are you commenting on > gfc_error ( > "%s argument at %L must be a scalar %s vari

[PATCH] libstdc++: Use UTF-32BE as wide encoding for big-endian machines [PR119725]

2025-04-13 Thread Tomasz Kamiński
This changes the `dg-options` line so UTF-32 with byte order native to the machine is used as wide encoding, avoiding. We still do not handle mismatch in the byte order of the Unicode encodings (UTF32-BE on little-endian machines). This would require larger changes, as for example `unicode-data.h`

Re: [PATCH] APX: Don't use red-zone with APX and no caller-saved registers

2025-04-13 Thread Hongtao Liu
On Mon, Apr 14, 2025 at 7:36 AM H.J. Lu wrote: > > Don't use red-zone when there are no caller-saved registers and APX is > enabled since 128-byte red-zone is too small for 31 GPRs. > > gcc/ > > PR target/119784 > * config/i386/i386.cc (ix86_using_red_zone): Don't use red-zone >

Re: [PATCH] doc: Document lack of SAHF support on Apple Rosetta 2 [PR119781]

2025-04-13 Thread Andrew Pinski
On Sun, Apr 13, 2025 at 9:30 AM Dimitri John Ledkov wrote: > > Apple Rosetta 2 x86-64 emulator is known to not support SAHF > instruction, which otherwise might be generated when targetting > -march=x86-64-v2. This seems out of place. Since we don't document all of the CPUs that implement v2/v3 h

[PATCH] doc: Document lack of SAHF support on Apple Rosetta 2 [PR119781]

2025-04-13 Thread Dimitri John Ledkov
Apple Rosetta 2 x86-64 emulator is known to not support SAHF instruction, which otherwise might be generated when targetting -march=x86-64-v2. PR target/119781 gcc/ChangeLog: * doc/invoke.texi: Document lack of SAHF on Apple Rosetta 2. Signed-off-by: Dimitri John Ledkov --- gc

Re: [PATCH] c++: wrong targs printed in hard satisfaction error [PR99214]

2025-04-13 Thread Patrick Palka
Alternatively, rather than passing the most general template + args to push_tinst_level, we can pass the partially instantiated template + innermost args via just: gcc/cp/ChangeLog: * constraint.cc (satisfy_declaration_constraints): Pass the original T and ARGS to push_tinst_level

Re: [Fortran, Patch, Teams, 0/5] Improve on Fortran 2018 teams support

2025-04-13 Thread Jerry D
On 4/10/25 5:59 AM, Andre Vehreschild wrote: Hi all, I again have a series of patches. This time to improve the teams support in gfortran. 1/5: Improves/Unifies handling of STAT= and ERRMSG= handling, which is part of all TEAM statements. I wanted to prevent repeating myself over and over so I

Re: [PATCH] [riscv] vec_dup immediate constants in pred_broadcast expand [PR118182]

2025-04-13 Thread Alexandre Oliva
On Apr 12, 2025, Jeff Law wrote: >> for gcc/ChangeLog >> PR target/118182 >> * config/riscv/vector.md (@pred_broadcast): Expand to >> _zero and _imm variants without vec_duplicate. > I'd said this should tend to wait for gcc-16 since it doesn't fix a > regression. That works for me. Technicall

Re: [PATCH] rx: avoid adding setpsw for rx_cmpstrn when len is const

2025-04-13 Thread Keith Packard
> Were you able to actually trigger cases where operands[3] is a > CONST_INT? Again, it's not entirely clear if that can actually happen > given the operand predicates on the cmpstrnsi expander. (sorry for the long delay; I'm just getting back to reviewing my pending GCC patches) Good call. N

[PATCH] [PR119765] testsuite: adjust amd64-abi-9.c to check both ms and sysv ABIs

2025-04-13 Thread Peter Damianov
This test was failing because it was checking that eax was being cleared. For sysv abi, eax contains the number of XMM registers used in the call, but msabi just passes the float arguments twice, both in xmm and general purpose registers. This patch adds tests for both sysv and msabi functions be

[PATCH] rx: avoid adding setpsw for rx_cmpstrn when len is const

2025-04-13 Thread Keith Packard
The only pattern for which the len may be zero is cmpstrsni. The other pattern using rx_cmpstrn is cmpstrsi for which len is a constant -1, so we'll be moving the setpsw instructions from rx_cmpstrn to cmpstrnsi as follows: 1. Adjust the predicate on the length operand from "register_operand"

Re: [PATCH v15] ada: fix timeval timespec on 32 bits archs with 64 bits time_t [PR114065]

2025-04-13 Thread Nicolas Boulenguez
Hello. Please skip v14 and review v15. A diff with v13 is attached. The full v15 is at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114065, but my SMTP server refused them as attachments. Patch 1 implements the new Ada.Calendar_Conversions.To_{Duration,Struct_Timespec}64 with System.C_Time, as alr

Re: [PATCH] cobol: Initialize regmatch_t portably [PR119217]

2025-04-13 Thread Rainer Orth
Hi Jakub, > On Fri, Apr 11, 2025 at 10:50:25AM +0200, Rainer Orth wrote: >> 2025-04-08 Rainer Orth >> >> gcc/cobol: >> PR cobol/119217 >> * dts.h (csub_match): Initialize rm_so, rm_eo fields explicitly. >> > >> # HG changeset patch >> # Parent 6f227ddea0046a0164509bdb8069e9618

[PATCH] APX: Don't use red-zone with APX and no caller-saved registers

2025-04-13 Thread H.J. Lu
Don't use red-zone when there are no caller-saved registers and APX is enabled since 128-byte red-zone is too small for 31 GPRs. gcc/ PR target/119784 * config/i386/i386.cc (ix86_using_red_zone): Don't use red-zone with APX and no caller-saved registers. gcc/testsuite/

[PATCH] Revert documents from r11-344-g0fec3f62b9bfc0

2025-04-13 Thread liuhongt
Look like those operand modifiers are only for internal usage in .md files, so for simplicity, I'll just remove them from extend.texi. Ready push to trunk. gcc/ChangeLog: PR documentation/108134 * doc/extend.texi: Remove documents from r11-344-g0fec3f62b9bfc0. --- gcc/doc/extend

Re: [PATCH] cobol: Avoid conflict with OVERFLOW in system headers [PR119217]

2025-04-13 Thread Rainer Orth
Hi Simon, > >> Shouldn't that be >> >> --- a/gcc/cobol/parse.y >> +++ b/gcc/cobol/parse.y >> @@ -337,7 +337,7 @@ >> %token INVALID >> %token NUMBER NEGATIVE >> %token NUMSTR"numeric literal" >> -%token OVERFLOW >> +%token OVERFLOW_kw "OVERFLOW" >>

Re: [PATCH] Locality cloning pass (was: Introduce -flto-partition=locality)

2025-04-13 Thread Jan Hubicka
> +@opindex fipa-reorder-for-locality > +@item -fipa-reorder-for-locality > +Group call chains close together in the binary layout to improve code code > +locality. This option is incompatible with an explicit > +@option{-flto-partition=} option since it enforces a custom partitioning > +scheme.

[PATCH 2/2] s390: Add z17 scheduler description

2025-04-13 Thread Stefan Schulze Frielinghaus
gcc/ChangeLog: * config/s390/s390.cc: Add z17 scheduler description. * config/s390/s390.h: Ditto. * config/s390/s390.md: Ditto. * config/s390/9175.md: New file. --- gcc/config/s390/9175.md | 316 gcc/config/s390/s390.cc | 3

[PATCH 1/2] s390: Support z17 processor name

2025-04-13 Thread Stefan Schulze Frielinghaus
The recently announced IBM z17 processor implements the architecture already supported as arch15. This patch adds support for z17 as an alternative architecture name for arch15. gcc/ChangeLog: * common/config/s390/s390-common.cc: Rename arch15 to z17. * config.gcc: Add z17.

Re: [patch, Fortran] Fix PR 119669

2025-04-13 Thread Thomas Koenig
Am 11.04.25 um 19:15 schrieb Harald Anlauf: There is a duplicate "and and" here: + gfc_error ("Mismatch between subroutine and and " +    "function at %L", &actual->where); OK with this fixed. Ah, yes :-) Thanks for the patch! Commit

Re: [PATCH] Add std::deque shrink_to_fit test

2025-04-13 Thread François Dumont
On 11/04/2025 08:36, Tomasz Kaminski wrote: On Thu, Apr 10, 2025 at 10:47 PM Jonathan Wakely wrote: On 10/04/25 22:36 +0200, François Dumont wrote: >After running the test with -fno-exceptions option we rather need this >patch. > >Ok to commit ? > >François

[PATCH] c++: Prune lambda captures from more places [PR119755]

2025-04-13 Thread Nathaniel Shead
Bootstrapped and regtested on x86_64-pc-linux-gnu, OK for trunk? -- >8 -- Currently, pruned lambda captures are still leftover in the function's BLOCK and topmost BIND_EXPR; this doesn't cause any issues for normal compilation, but does break modules streaming as we try to reconstruct a FIELD_DEC

Polish 'dg-output-file' test logs (was: [PATCH] testsuite: Add support for dg-output-file directive)

2025-04-13 Thread Thomas Schwinge
Hi! On 2025-03-18T14:54:23+0100, Jakub Jelinek wrote: > The following patch offers [...] dg-output-file > directive where one can supply a text file with expected output > --- gcc/doc/sourcebuild.texi.jj 2025-03-11 09:18:21.750133577 +0100 > +++ gcc/doc/sourcebuild.texi 2025-03-18 14:41:5

New Swedish PO file for 'gcc' (version 15.1-b20250406)

2025-04-13 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Swedish team of translators. The file is available at: https://translationproject.org/latest/gcc/sv.po (This file, 'gcc-15.1-b20250406.sv.po'