On 6/13/25 1:36 PM, Patrick Palka wrote:
On Fri, 13 Jun 2025, Jason Merrill wrote:
Tested x86_64-pc-linux-gnu, any comments? Bikeshedding?
-- 8< --
We already error about a type definition causing a concept check to change
value, but it would be useful to diagnose this for other SFINAE conte
Hi,
> On 13 Jun 2025, at 7:21 pm, Jan Hubicka wrote:
>
> External email: Use caution opening links or attachments
>
>
>> From: Dhruv Chawla
> Hi,
>>
>> For reasons explained in the patch, this patch prevents the loss of profile
>> information when inlining occurs in the profiled binary but n
On Fri, 13 Jun 2025, Pietro Monteiro wrote:
> > Adding this \ at end of file looks suspect.
> >
> > OK with that ChangeLog entry fixed and the stray \ at end of file removed
> > (assuming the same output files are still generated after that change).
>
> I applied the patch on top of trunk (), re
On Fri, Jun 13, 2025, at 2:02 PM, Joseph Myers wrote:
> On Wed, 21 May 2025, Pietro Monteiro wrote:
>
>> lto-plugin/ChangeLog:
>>
>> * configure: Regenerate.
>> * configure.ac: Replace AC_CANONICAL_SYSTEM with AC_CANONICAL_SYSTEM.
>
> That ChangeLog entry is clearly incorrect.
Sorry. It
On Wed, Jun 04, 2025 at 09:18:45PM +0800, yxj-github-437 wrote:
> > This line seems wrongly indented, should be only two spaces more
> > than the if line:
> >
> > if (check_for_bare_parameter_packs (expression))
> >expression = error_mark_node;
> >
> >
> > The patch LGTM otherwise, thanks.
>
On Thu, Jun 12, 2025 at 05:39:26PM +, Joseph Myers wrote:
> On Wed, 11 Jun 2025, Marek Polacek wrote:
>
> > This patch adds the C23 Support in GCC table (compiler features only).
> >
> > While creating it, I've consulted Annex M.2, our own changes.html,
> > Joseph's "ISO C23 support in the GN
gcc/c/ChangeLog:
* c-typeck.cc (build_counted_by_ref): Handle pointers with counted_by.
(build_access_with_size_for_counted_by): Likewise.
gcc/ChangeLog:
* tree-object-size.cc (access_with_size_object_size): Handle pointers
with counted_by.
(collect_object
Current array bound checker only instruments ARRAY_REF, and the INDEX
information is the 2nd operand of the ARRAY_REF.
When extending the array bound checker to pointer references with
counted_by attributes, the hardest part is to get the INDEX of the
corresponding array ref from the offset comput
Hi,
This is the 5th version of the patch set to extend "counted_by" attribute
to pointer fields of structures.
compared to the 4rd version:
https://gcc.gnu.org/pipermail/gcc-patches/2025-May/683538.html
https://gcc.gnu.org/pipermail/gcc-patches/2025-May/683539.html
https://gcc.gnu.org/pipermail
For example:
struct PP {
size_t count2;
char other1;
char *array2 __attribute__ ((counted_by (count2)));
int other2;
} *pp;
specifies that the "array2" is an array that is pointed by the
pointer field, and its number of elements is given by the field
"count2" in the same structure.
gcc/c-
Hello,
On 13/06/2025 15:02, Jonathan Wakely wrote:
OK thanks
Committed as r16-1501-gf6f4efdf397269.
Thanks,
--
Giuseppe D'Angelo
smime.p7s
Description: S/MIME Cryptographic Signature
On Wed, 21 May 2025, Pietro Monteiro wrote:
> lto-plugin/ChangeLog:
>
> * configure: Regenerate.
> * configure.ac: Replace AC_CANONICAL_SYSTEM with AC_CANONICAL_SYSTEM.
That ChangeLog entry is clearly incorrect.
> diff --git a/config/asmcfi.m4 b/config/asmcfi.m4
> index a725aa11de4.
On Fri, 13 Jun 2025, Jason Merrill wrote:
> Tested x86_64-pc-linux-gnu, any comments? Bikeshedding?
>
> -- 8< --
>
> We already error about a type definition causing a concept check to change
> value, but it would be useful to diagnose this for other SFINAE contexts as
> well; the memoization p
Tested x86_64-pc-linux-gnu, any comments? Bikeshedding?
-- 8< --
We already error about a type definition causing a concept check to change
value, but it would be useful to diagnose this for other SFINAE contexts as
well; the memoization problem also affects templates. So
-Wsfinae-incomplete re
On 6/6/25 3:21 AM, Alexandre Oliva wrote:
On Jun 6, 2025, Alexandre Oliva wrote:
Now, since lra_update_fp2sp_elimination checks that
!elimination_fp2sp_occured_p, we *could* disable the fp2sp elimination,
if it's selected, right away, so that it is not applied after we've
disabled it, and t
On Fri, 13 Jun 2025, Gábor Németh wrote:
> > > A new option is added to warn if floating point literals have non-standard
> > > suffices (currently Q and W) in pedantic mode. The option is ON by
> > > default.
> >
> > I don't think we should turn this off when building GCC itself (as opposed
> >
A new option is added to warn if floating point literals have non-standard
suffices (currently Q and W) in pedantic mode. The option is ON by default.
I don't think we should turn this off when building GCC itself (as opposed
to target libraries). Host floating-point should only be used for lim
On 6/13/25 11:22, Alex Coplan wrote:
> External email: Use caution opening links or attachments
>
>
> Hi Remi,
>
> On 12/06/2025 17:02, Richard Sandiford wrote:
>> Remi Machet writes:
>>> Add an optimization to aarch64 SIMD converting mvn+shrn into mvni+subhn
>>> which
>>> allows for better o
Alex Coplan writes:
> Hi Remi,
>
> On 12/06/2025 17:02, Richard Sandiford wrote:
>> Remi Machet writes:
>> > + "TARGET_SIMD"
>> > + "#"
>> > + "&& true"
>> > + [(const_int 0)]
>> > +{
>> > + rtx tmp;
>> > + if (can_create_pseudo_p ())
>> > + tmp = gen_reg_rtx (mode);
>> > + else
>> > +
Hi Remi,
On 12/06/2025 17:02, Richard Sandiford wrote:
> Remi Machet writes:
> > Add an optimization to aarch64 SIMD converting mvn+shrn into mvni+subhn
> > which
> > allows for better optimization when the code is inside a loop by using a
> > constant.
It can be helpful for reviewers to show t
Hi,
All macOS SDK since at least macOS 10.9, and until macOS 10.12
(included), feature these lines in :
/* DO NOT REMOVE THIS COMMENT: fixincludes needs to see:
* __gnuc_va_list and include */
The clear intent (and effect) was to bypass gcc’s stdio_stdarg_h
fixinclude.
However, since macOS 10
On 6/11/25 1:12 PM, Iain Sandoe wrote:
On 11 Jun 2025, at 17:53, Jason Merrill wrote:
On 6/9/25 3:54 PM, Iain Sandoe wrote:
I was planning to apply this as obvious - but it is needed for the
next patch to be posted - so noting here now. I discussed with one
of the original coroutines paper
On Fri, Jun 13, 2025 at 02:12:44PM +, Kyrylo Tkachov wrote:
> Hi Spencer,
>
> Thanks for the patch.
>
> > On 13 Jun 2025, at 14:46, Spencer Abson wrote:
> >
> > Add the missing combiner patterns for folding NOT+PTEST to NOTS when
> > they share the same GP.
> >
>
> I guess GP here means “
Addressed the most of comments and tried to refactor the
riscv_expand_conditional_move() to some extent.
No regressions are found for "runtest --tool gcc
--target_board='riscv-sim/-mabi=lp64d/-mtune=mips-p8700/-O2 ' riscv.exp"
*config/riscv/riscv-cores.def(RISCV_CORE):Updated the suppor
On Fri, 13 Jun 2025, Tomasz Kaminski wrote:
>
>
> On Fri, Jun 13, 2025 at 3:54 PM Patrick Palka wrote:
> On Fri, 13 Jun 2025, Tomasz Kaminski wrote:
>
> >
> >
> > On Thu, Jun 12, 2025 at 9:05 PM Patrick Palka
> wrote:
> > On Thu, 12 Jun 2025, Patrick Palka
On Fri, Jun 13, 2025 at 4:46 PM Patrick Palka wrote:
> On Fri, 13 Jun 2025, Tomasz Kaminski wrote:
>
> >
> >
> > On Fri, Jun 13, 2025 at 3:54 PM Patrick Palka wrote:
> > On Fri, 13 Jun 2025, Tomasz Kaminski wrote:
> >
> > >
> > >
> > > On Thu, Jun 12, 2025 at 9:05 PM Patr
On Fri, Jun 13, 2025 at 3:54 PM Patrick Palka wrote:
> On Fri, 13 Jun 2025, Tomasz Kaminski wrote:
>
> >
> >
> > On Thu, Jun 12, 2025 at 9:05 PM Patrick Palka wrote:
> > On Thu, 12 Jun 2025, Patrick Palka wrote:
> >
> > > On Thu, 12 Jun 2025, Jonathan Wakely wrote:
> > >
> >
On 13/06/25 09:54 -0400, Patrick Palka wrote:
On Fri, 13 Jun 2025, Tomasz Kaminski wrote:
On Thu, Jun 12, 2025 at 9:05 PM Patrick Palka wrote:
On Thu, 12 Jun 2025, Patrick Palka wrote:
> On Thu, 12 Jun 2025, Jonathan Wakely wrote:
>
> >
> >
> > On Thu, 12
Hi Spencer,
Thanks for the patch.
> On 13 Jun 2025, at 14:46, Spencer Abson wrote:
>
> Add the missing combiner patterns for folding NOT+PTEST to NOTS when
> they share the same GP.
>
I guess GP here means “governing predicate”?
GP usually means “General Purpose (register)” in aarch64 so it’d
On Fri, 13 Jun 2025, Tomasz Kaminski wrote:
>
>
> On Thu, Jun 12, 2025 at 9:05 PM Patrick Palka wrote:
> On Thu, 12 Jun 2025, Patrick Palka wrote:
>
> > On Thu, 12 Jun 2025, Jonathan Wakely wrote:
> >
> > >
> > >
> > > On Thu, 12 Jun 2025, 16:56 Patrick Palk
Add an optimization to aarch64 SIMD converting mvn+shrn into mvni+subhn
which
allows for better optimization when the code is inside a loop by using a
constant.
Bootstrapped and regtested on aarch64-linux-gnu.
Signed-off-by: Remi Machet
gcc/ChangeLog:
* config/aarch64/aarch64-simd.md
On Fri, 13 Jun 2025 at 14:17, Tomasz Kamiński wrote:
>
> Similarly to issue reported for %c in PR117214, the format string for locale
> specific time (%r, %X) and date (%x) representations may contain specifiers
> not accepted by chrono-spec, leading to exception being thrown. This
> happened for
On Fri, 13 Jun 2025 at 10:38, Daniel Krügler wrote:
>
> Am Fr., 13. Juni 2025 um 11:33 Uhr schrieb Jonathan Wakely
> :
>>
>> The std::uninitialized_{value,default}_construct{,_n} algorithms should
>> be able to create arrays, but that currently fails because when an
>> exception happens they clea
Similarly to issue reported for %c in PR117214, the format string for locale
specific time (%r, %X) and date (%x) representations may contain specifiers
not accepted by chrono-spec, leading to exception being thrown. This
happened for following conversion specifier and locale combinations:
* %r, %
> -Original Message-
> From: Jan Hubicka
> Sent: Monday, April 21, 2025 6:35 PM
> To: H.J. Lu
> Cc: gcc-patches@gcc.gnu.org; Liu, Hongtao ;
> ubiz...@gmail.com
> Subject: Re: [PATCH v2] x86: Update memcpy/memset inline strategies for -
> mtune=generic
>
> > On Mon, Apr 21, 2025 at 7:24
Jakub Jelinek writes:
> On Fri, Jun 13, 2025 at 10:05:14AM +0200, Jakub Jelinek wrote:
>> On Fri, Jun 13, 2025 at 08:52:55AM +0100, Richard Sandiford wrote:
>> > > 2025-06-12 Jakub Jelinek
>> > >
>> > > * cfgexpand.cc (construct_init_block): If first_block isn't BB_RTL,
>> > > has any PHI nod
On Fri, 13 Jun 2025 at 10:55, Giuseppe D'Angelowrote:
>
> Hi,
>
> Attached is a workaround for
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=120644
>
> It looks it's a C++ frontend bug, this commit just silences the
> regression in the modules testsuite that have been observed after the
> introd
Jiawei writes:
> This patch adds a new simplification rule to `simplify-rtx.cc` that
> handles a common bit manipulation pattern involving a single-bit set
> and clear followed by XOR.
>
> The transformation targets RTL of the form:
>
> (xor (and (rotate (~1) A) B) (ashift 1 A))
>
> which is sem
Add the missing combiner patterns for folding NOT+PTEST to NOTS when
they share the same GP.
gcc/ChangeLog:
* config/aarch64/aarch64-sve.md (*one_cmpl3_cc): New
combiner pattern.
(*one_cmpl3_ptest): Likewise.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/sve/acle
On Fri, Jun 13, 2025 at 10:05:14AM +0200, Jakub Jelinek wrote:
> On Fri, Jun 13, 2025 at 08:52:55AM +0100, Richard Sandiford wrote:
> > > 2025-06-12 Jakub Jelinek
> > >
> > > * cfgexpand.cc (construct_init_block): If first_block isn't BB_RTL,
> > > has any PHI nodes and false_edge->dest_idx
Thank you, applied to trunk!
--Philipp.
On Fri, 13 Jun 2025 at 09:30, Konstantinos Eleftheriou
wrote:
>
> On Thu, Jun 12, 2025 at 5:20 PM Jeff Law wrote:
> >
> >
> >
> > On 6/10/25 1:09 AM, Konstantinos Eleftheriou wrote:
> > > Testcase pr119160.c fails with symbol referencing errors for
> > >
From: Yunze Zhu
This commit add support for xtheadvector-specific fault-only-first segment
load/store intrinsics with b/h/w suffix. We also defined enum to be used
in thead-vector.md
https://github.com/XUANTIE-RV/thead-extension-spec/pull/66
V2:
Change to reuse existed thead function base th_lo
> On 13/06/25 14:51, Jan Hubicka wrote:
> > External email: Use caution opening links or attachments
> >
> >
> > > From: Dhruv Chawla
> > Hi,
> > >
> > > For reasons explained in the patch, this patch prevents the loss of
> > > profile
> > > information when inlining occurs in the profiled bin
From: Yunze Zhu
This commit add support for xtheadvector-specific indexed segment load/store
intrinsics with b/h/w suffix. We also defined enum to be used in
thead-vector.md
https://github.com/XUANTIE-RV/thead-extension-spec/pull/66
V2:
Change to reuse existed thead function base th_loadstore_w
From: Yunze Zhu
This commit add support for xtheadvector-specific strided segment load/store
intrinsics with b/h/w suffix. We also defined enum to be used in
thead-vector.md
https://github.com/XUANTIE-RV/thead-extension-spec/pull/66
V2:
Change to reuse existed thead function base th_loadstore_w
From: Yunze Zhu
This commit add support for xtheadvector-specific unit-stride segment
load/store intrinsics with b/h/w suffix. We also defined enum to be used
in thead-vector.md
https://github.com/XUANTIE-RV/thead-extension-spec/pull/66
V2:
Change to reuse existed thead function base th_loadsto
Thank you both for the tips and information:
* In that case I can update the auxiliary files after mdspan.
* I use gcc-commit-mklog, but during rebasing / fixups, I edit
the changelog manually (and mistakes creep in). Just tried
gcc-verify it found a mistake in a patch I was preparing and
libstdc++-v3/ChangeLog:
* include/std/mdspan (default_accessor): New class.
* src/c++23/std.cc.in: Register default_accessor.
* testsuite/23_containers/mdspan/accessors/default.cc: New test.
Signed-off-by: Luc Grosheintz
---
libstdc++-v3/include/std/mdspan
This is a rebased and retested version of:
https://gcc.gnu.org/pipermail/libstdc++/2025-June/061936.html
The test has been moved to a subdirectory `accessors`.
Luc Grosheintz (1):
libstdc++: Implement default_accessor from mdspan.
libstdc++-v3/include/std/mdspan | 26
l
This patch adds a new simplification rule to `simplify-rtx.cc` that
handles a common bit manipulation pattern involving a single-bit set
and clear followed by XOR.
The transformation targets RTL of the form:
(xor (and (rotate (~1) A) B) (ashift 1 A))
which is semantically equivalent to:
B |
> -Original Message-
> From: Uros Bizjak
> Sent: Thursday, June 12, 2025 5:05 PM
> To: Cui, Lili
> Cc: gcc-patches@gcc.gnu.org; Liu, Hongtao ;
> richard.guent...@gmail.com; Michael Matz
> Subject: Re: [PATCH V2] x86: Enable separate shrink wrapping
>
> On Thu, Jun 12, 2025 at 10:58 AM
LGTM, thanks.
It's interesting that having deduced the return type caused an issue.
On Fri, Jun 13, 2025 at 11:56 AM Giuseppe D'Angelo <
giuseppe.dang...@kdab.com> wrote:
> Hi,
>
> Attached is a workaround for
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=120644
>
> It looks it's a C++ fronten
Hi,
Attached is a workaround for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=120644
It looks it's a C++ frontend bug, this commit just silences the
regression in the modules testsuite that have been observed after the
introduction of range support for optional in r16-1487-g774ae8734f0e19.
T
On 28/04/2025 16:33, Alex Coplan wrote:
> On 15/04/2025 19:48, Jan Hubicka wrote:
> > Hi,
> > > gcc/ChangeLog:
> > >
> > > PR tree-optimization/117790
> > > * tree-vect-loop.cc (scale_profile_for_vect_loop): Use
> > > scale_loop_profile_hold_exit_counts instead of scale_loop_profile. Drop
>
On 13/06/25 14:51, Jan Hubicka wrote:
External email: Use caution opening links or attachments
From: Dhruv Chawla
Hi,
For reasons explained in the patch, this patch prevents the loss of profile
information when inlining occurs in the profiled binary but not in the
auto-profile pass as a de
On Fri, Jun 13, 2025 at 11:31 AM Jonathan Wakely wrote:
> The std::uninitialized_{value,default}_construct{,_n} algorithms should
> be able to create arrays, but that currently fails because when an
> exception happens they clean up using std::_Destroy and in C++17 that
> doesn't support destroyi
Am Fr., 13. Juni 2025 um 11:33 Uhr schrieb Jonathan Wakely <
jwak...@redhat.com>:
> The std::uninitialized_{value,default}_construct{,_n} algorithms should
> be able to create arrays, but that currently fails because when an
> exception happens they clean up using std::_Destroy and in C++17 that
>
On Wed, Jun 11, 2025 at 11:36 AM Jonathan Wakely wrote:
> Clarify that FSF copyright notices in tests are incorrect for
> contributions under DCO terms. Clarify the sentence about copying
> existing tests to clarify that it is only referring to copying the code
> in the test file, rather than jus
The std::uninitialized_{value,default}_construct{,_n} algorithms should
be able to create arrays, but that currently fails because when an
exception happens they clean up using std::_Destroy and in C++17 that
doesn't support destroying arrays. (For C++20 and later, std::destroy
does handle destroyi
> From: Dhruv Chawla
Hi,
>
> For reasons explained in the patch, this patch prevents the loss of profile
> information when inlining occurs in the profiled binary but not in the
> auto-profile pass as a decision. As an example, for this code:
I was wondering about this problem too
> - Annotation
From: Dhruv Chawla
This patch modifies afdo_set_bb_count to propagate profile information
to outline copies of functions if they are not inlined. This information
gets lost otherwise.
Signed-off-by: Dhruv Chawla
gcc/ChangeLog:
* gcc/auto-profile.cc (count_info): Adjust comments.
On Fri, Jun 13, 2025 at 08:52:55AM +0100, Richard Sandiford wrote:
> > 2025-06-12 Jakub Jelinek
> >
> > * cfgexpand.cc (construct_init_block): If first_block isn't BB_RTL,
> > has any PHI nodes and false_edge->dest_idx before redirection is
> > different from make_single_succ_edge re
Am Fr., 13. Juni 2025 um 09:33 Uhr schrieb Tomasz Kamiński <
tkami...@redhat.com>:
> As pointed out by Daniel Krügler we do not need to use reserved name
> in tests.
>
> libstdc++-v3/ChangeLog:
>
> * testsuite/23_containers/vector/bool/format.cc: Replaced _CharT
> with CharT.
>
Jakub Jelinek writes:
> Hi!
>
> Andrew ran some extra ranger checking during bootstrap and found one more
> case (though much rarer than the GIMPLE_COND case).
>
> Seems on fold-const.cc (native_encode_expr) we end up with bb 2, ENTRY
> bb successor, having PHI nodes (usually there is some bb in b
As pointed out by Daniel Krügler we do not need to use reserved name
in tests.
libstdc++-v3/ChangeLog:
* testsuite/23_containers/vector/bool/format.cc: Replaced _CharT
with CharT.
* testsuite/std/format/debug.cc: Likewise.
* testsuite/std/format/ranges/adaptors.cc:
On Thu, Jun 12, 2025 at 5:20 PM Jeff Law wrote:
>
>
>
> On 6/10/25 1:09 AM, Konstantinos Eleftheriou wrote:
> > Testcase pr119160.c fails with symbol referencing errors for
> > `__cyg_profile_func_enter` and `__cyg_profile_func_exit` on non-glibc
> > systems.
> >
> > This patch adds empty definiti
On Thu, Jun 12, 2025 at 7:33 PM Patrick Palka wrote:
> On Thu, 12 Jun 2025, Patrick Palka wrote:
>
> > Changes in v4:
> > * Don't pass a projection function throughout the helpers, instead
> > create a pass a composite predicate via __make_comp_proj.
>
LGTM.
> >
> > -- >8 --
> >
> > As wit
On Thu, Jun 12, 2025 at 6:53 PM Patrick Palka wrote:
> Changes in v4:
> * Don't pass a projection function throughout the helpers, instead
> create a pass a composite predicate via __make_comp_proj.
>
LGTM.
>
> -- >8 --
>
> ranges::push_heap, ranges::pop_heap, ranges::make_heap and
> range
68 matches
Mail list logo
