[PATCH v9 3/3] c: Add __elementsof__ operator

2024-08-14 Thread Alejandro Colomar
This operator is similar to sizeof but can only be applied to an array, and returns its number of elements. FUTURE DIRECTIONS: - We should make it work with array parameters to functions, and somehow magically return the number of elements of the array, regardless of it being really a poin

Re: [PATCH v2] aarch64: Improve popcount for bytes [PR113042]

2024-08-14 Thread Richard Sandiford
Andrew Pinski writes: > For popcount for bytes, we don't need the reduction addition > after the vector cnt instruction as we are only counting one > byte's popcount. > This changes the popcount extend to cover all ALLI rather than GPI. > > Changes since v1: > * v2 - Use ALLI iterator and combine

Re: [PATCH-1v4] Value Range: Add range op for builtin isinf

2024-08-14 Thread Jeff Law
On 8/14/24 12:38 PM, Sam James wrote: Vineet Gupta writes: Ping - looks like this is blocking the patches for builtin_isnormal and builtin_isfinite ! See https://inbox.sourceware.org/gcc-patches/d9459db0-7301-40f6-a3cf-077017b8c...@gmail.com/. It appears to be approved. (Please also

Re: [PATCH] Re-add calling emit_clobber in lower-subreg.cc's resolve_simple_move.

2024-08-14 Thread Richard Sandiford
Jeff Law writes: > On 8/12/24 10:12 AM, Xianmiao Qu wrote: >> The previous patch: >> https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=d8a6945c6ea22efa4d5e42fe1922d2b27953c8cd >> aimed to eliminate redundant MOV instructions by removing calling >> emit_clobber in lower-subreg.cc's resolve_simple_move.

Re: [PATCH] Re-add calling emit_clobber in lower-subreg.cc's resolve_simple_move.

2024-08-14 Thread Jeff Law
On 8/14/24 3:53 PM, Richard Sandiford wrote: FWIW, I think the work to add a df subreg liveness tracking problem and use it in LRA/IRA would solve the live range problem without needing a clobber. I wonder how that's going? In my last review I suggested a change in representation (a single

Re: [PATCH v4] Target-independent store forwarding avoidance.

2024-08-14 Thread Jeff Law
On 7/8/24 6:46 AM, Manolis Tsamis wrote: This pass detects cases of expensive store forwarding and tries to avoid them by reordering the stores and using suitable bit insertion sequences. For example it can transform this: [ ... ] So FYI. Clean bootstrap and regression test on m68k, sh4, sh

Re: [PATCH] libcpp, c-family, v2: Add (dumb) C23 N3017 #embed support [PR105863]

2024-08-14 Thread Joseph Myers
On Tue, 18 Jun 2024, Jakub Jelinek wrote: > The following patch implements the C23 N3017 "#embed - a scannable, > tooling-friendly binary resource inclusion mechanism" paper. Some initial comments, not yet reviewed the whole patch. > The patch uses --embed-dir= option that clang plans to add abo

[to-be-committed][RISC-V][PR target/116282] Stabilize pattern conditions

2024-08-14 Thread Jeff Law
So as expected the core problem with target/116282 is that the cost of certain constant synthesis cases varied depending on whether or not we're allowed to generate new pseudos or not. That in turn meant that in obscure cases an insn might change from recognizable to unrecognizable and trigger

[PATCH 1/2] Makefile.tpl: drop leftover intermodule cruft

2024-08-14 Thread Sam James
intermodule supported was dropped in r0-103106-gde6ba7aee152a0 with some remaining bits for Fortran removed in r14-1696-gecc96eb5d2a0e5. Remove some small leftovers. * Makefile.in: Regenerate. * Makefile.tpl (STAGE1_CONFIGURE_FLAGS: Remove --disable-intermodule. --- Makefile.in

[PATCH 2/2] Makefile.tpl: fix whitespace in licence header

2024-08-14 Thread Sam James
* Makefile.tpl: Fix whitespace. --- Makefile.tpl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Makefile.tpl b/Makefile.tpl index cbb3c6789dcf..da38dca697ad 100644 --- a/Makefile.tpl +++ b/Makefile.tpl @@ -13,12 +13,12 @@ in # it under the terms of the GNU General

Re: [RFC] libstdc++: Replace Ryu with Teju Jagua for float.

2024-08-14 Thread Patrick Palka
On Mon, 5 Aug 2024, Cassio Neri wrote: > Implement the template function teju_jagua which finds the shortest > representation of a floating-point number. The floating-point type is a > template parameter and the implementation is generic enough to handle all > floating-point types of interest, nam

Re: [PATCH v4] diagnostics: Follow DECL_ORIGIN in lhd_print_error_function [PR102061]

2024-08-14 Thread Peter0x44
On 2024-08-11 04:42, Peter Damianov wrote: Currently, if a warning references a cloned function, the name of the cloned function will be emitted in the "In function 'xyz'" part of the diagnostic, which users aren't supposed to see. This patch follows the DECL_ORIGIN link to get the name of the

Re: [PATCH 1/2] Makefile.tpl: drop leftover intermodule cruft

2024-08-14 Thread Sam James
Sam James writes: > intermodule supported was dropped in r0-103106-gde6ba7aee152a0 with some > remaining bits for Fortran removed in r14-1696-gecc96eb5d2a0e5. > > Remove some small leftovers. > > * Makefile.in: Regenerate. > * Makefile.tpl (STAGE1_CONFIGURE_FLAGS: Remove --disable-int

Re: [PATCH] gm2: fix bad programming practice warning

2024-08-14 Thread Gaius Mulley
Wilken Gottwalt writes: > On Wed, 24 Jul 2024 12:09:46 +0200 > Wilken Gottwalt wrote: > >> Fix identifier names to be too similar to Modula-2 keywords and causing >> warnings coming from Modula-2's own libraries. >> >> m2/m2iso/StdChans.mod:54:20: note: In implementation module ‘StdChans’: >> e

Re: [PATCH] gm2: fix bad programming practice identifier warning

2024-08-14 Thread Gaius Mulley
Wilken Gottwalt writes: > On Tue, 23 Jul 2024 09:58:56 +0200 > Wilken Gottwalt wrote: > >> Fix using keywords as identifiers to prevent warnings coming from >> Modula-2's own libraries. >> >> m2pim/DynamicStrings.mod:1358:27: note: In procedure ‘Slice’: the symbol >> name ‘end’ is legal as an i

RE: [PATCH 00/22] Support AVX10.2 ymm rounding

2024-08-14 Thread Jiang, Haochen
> -Original Message- > From: Haochen Jiang > Sent: Wednesday, August 14, 2024 5:02 PM > To: gcc-patches@gcc.gnu.org > Cc: Liu, Hongtao ; ubiz...@gmail.com > Subject: [PATCH 00/22] Support AVX10.2 ymm rounding > > Hi all, > > The initial patch for AVX10.2 has been merged this week. > > F

Re: [PATCH] libstdc++-v3: testsuite: Prune uncapitalized "in function" linker warning

2024-08-14 Thread Jacob Bachmeyer
Hans-Peter Nilsson wrote: (CC to the dejagnu project as a heads-up) Regtested cris-elf with a fresh newlib checkout where 2640 libstdc++-v3 tests otherwise fail due to the stubbed newlib _getentropy. Ok to commit? -- >8 -- Newer newlib trigger warnings about certain functions not implemented (

Re: [PATCH] libstdc++-v3: testsuite: Prune uncapitalized "in function" linker warning

2024-08-14 Thread Hans-Peter Nilsson
> Date: Wed, 14 Aug 2024 20:58:04 -0500 > From: Jacob Bachmeyer > Reply-To: jcb62...@gmail.com > Hans-Peter Nilsson wrote: > > (CC to the dejagnu project as a heads-up) > > > > Regtested cris-elf with a fresh newlib checkout where 2640 > > libstdc++-v3 tests otherwise fail due to the stubbed newl

Re: [PATCH] libstdc++-v3: testsuite: Prune uncapitalized "in function" linker warning

2024-08-14 Thread Jacob Bachmeyer
Hans-Peter Nilsson wrote: Date: Wed, 14 Aug 2024 20:58:04 -0500 From: Jacob Bachmeyer Reply-To: jcb62...@gmail.com [...] If this is correct, I will fix it in Git master. (The fix is trivial: change the "I" to "[Ii]" to accept both forms.) Yes, thanks; just don't forget to escape t

Re: [PATCH-1v4] Value Range: Add range op for builtin isinf

2024-08-14 Thread HAO CHEN GUI
Hi All, These three value range patches were committed as r15-2922, r15-2923 and r15-2924. Thanks Gui Haochen 在 2024/8/15 5:46, Jeff Law 写道: > > > On 8/14/24 12:38 PM, Sam James wrote: >> Vineet Gupta writes: >> >>> Ping - looks like this is blocking the patches for builtin_isnormal and >>

[PATCH] Optimize initialization of small padded objects

2024-08-14 Thread Alexandre Oliva
When small objects containing padding bits (or bytes) are fully initialized, we will often store them in registers, and setting bitfields and other small fields will attempt to preserve the uninitialized padding bits, which tends to be expensive. Zero-initializing registers, OTOH, tends to be che

[PATCH v3] aarch64: Improve popcount for bytes [PR113042]

2024-08-14 Thread Andrew Pinski
For popcount for bytes, we don't need the reduction addition after the vector cnt instruction as we are only counting one byte's popcount. This changes the popcount extend to cover all ALLI rather than GPI. Changes since v1: * v2 - Use ALLI iterator and combine all into one pattern. Add new

Re: [PATCH v2] aarch64: Improve popcount for bytes [PR113042]

2024-08-14 Thread Andrew Pinski
On Wed, Aug 14, 2024 at 2:21 PM Richard Sandiford wrote: > > Andrew Pinski writes: > > For popcount for bytes, we don't need the reduction addition > > after the vector cnt instruction as we are only counting one > > byte's popcount. > > This changes the popcount extend to cover all ALLI rather t

Re: [PATCH v2] i386: Fix some vex insns that prohibit egpr

2024-08-14 Thread Hongtao Liu
On Wed, Aug 14, 2024 at 4:23 PM Kong, Lingling wrote: > > > > -Original Message- > From: Kong, Lingling > Sent: Wednesday, August 14, 2024 4:20 PM > To: Kong, Lingling > Subject: [PATCH v2] i386: Fix some vex insns that prohibit egpr > > Although these vex insn have evex counterpart, but

<    1   2