Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Jeff Law via Gcc-patches
On 5/5/23 14:46, Jakub Jelinek wrote: On Fri, May 05, 2023 at 03:37:47PM +, Tamar Christina wrote: 2023-05-05 Jakub Jelinek * Makefile.in (check_p_numbers): Rename to one_to_, move earlier with helper variables also renamed. (MATCH_SPLUT_SEQ): Use $(wordlis

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Jakub Jelinek via Gcc-patches
On Fri, May 05, 2023 at 03:37:47PM +, Tamar Christina wrote: > > 2023-05-05 Jakub Jelinek > > > > * Makefile.in (check_p_numbers): Rename to one_to_, move > > earlier with helper variables also renamed. > > (MATCH_SPLUT_SEQ): Use $(wordlist 1,$(NUM_MATCH_SPLITS),$(one_to_999

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread David Edelsohn via Gcc-patches
On Fri, May 5, 2023 at 11:38 AM Tamar Christina wrote: > > -Original Message- > > From: Jakub Jelinek > > Sent: Friday, May 5, 2023 4:33 PM > > To: Tamar Christina > > Cc: Jeff Law ; David Edelsohn >; > > GCC Patches > > Subject: Re: [PA

RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Jakub Jelinek > Sent: Friday, May 5, 2023 4:33 PM > To: Tamar Christina > Cc: Jeff Law ; David Edelsohn ; > GCC Patches > Subject: Re: [PATCH 5/5] match.pd: Use splits in makefile and make > configurable. > > On Fri, May 05, 2

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Jakub Jelinek via Gcc-patches
On Fri, May 05, 2023 at 03:22:11PM +, Tamar Christina wrote: > > We require GNU make, so perhaps we could use something like $(wordlist > > 1,$(NUM_MATCH_SPLITS),$(check_p_numbers)) > > instead of > > $(shell seq 1 $(NUM_MATCH_SPLITS)) > > provided we move the check_p_numbers definition earlier

RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Jakub Jelinek > Sent: Friday, May 5, 2023 4:18 PM > To: Jeff Law > Cc: David Edelsohn ; Tamar Christina > ; GCC Patches > Subject: Re: [PATCH 5/5] match.pd: Use splits in makefile and make > configurable. > > On Fri, May 05,

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Jakub Jelinek via Gcc-patches
On Fri, May 05, 2023 at 09:04:16AM -0600, Jeff Law via Gcc-patches wrote: > On 5/5/23 08:59, David Edelsohn via Gcc-patches wrote: > > This patch has broken GCC bootstrap on AIX. It appears to rely upon, or > > complain about, the command "seq": > > > > /nasfarm/edelsohn/install/GCC12/bin/g++ -st

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Jeff Law via Gcc-patches
On 5/5/23 09:08, Tamar Christina wrote: -Original Message- From: Jeff Law Sent: Friday, May 5, 2023 4:04 PM To: David Edelsohn ; Tamar Christina Cc: GCC Patches Subject: Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable. On 5/5/23 08:59, David Edelsohn via

RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Jeff Law > Sent: Friday, May 5, 2023 4:04 PM > To: David Edelsohn ; Tamar Christina > > Cc: GCC Patches > Subject: Re: [PATCH 5/5] match.pd: Use splits in makefile and make > configurable. > > > > On 5/5/23 08:59,

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Jeff Law via Gcc-patches
On 5/5/23 08:59, David Edelsohn via Gcc-patches wrote: This patch has broken GCC bootstrap on AIX. It appears to rely upon, or complain about, the command "seq": /nasfarm/edelsohn/install/GCC12/bin/g++ -std=c++11 -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread David Edelsohn via Gcc-patches
This patch has broken GCC bootstrap on AIX. It appears to rely upon, or complain about, the command "seq": /nasfarm/edelsohn/install/GCC12/bin/g++ -std=c++11 -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wno-format -Wmiss

RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-05 Thread Tamar Christina via Gcc-patches
> This looks pretty reasonable to me. Are there any patches left in > this series that need review? I'm very much looking forward to > build time provements related to this patch, particularly for > targets that I bootstrap with qemu emulation -- we take multiple > hours

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-04 Thread Jeff Law via Gcc-patches
; ralf.wildenh...@gmx.de Subject: RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable. -Original Message- From: Jeff Law Sent: Sunday, April 30, 2023 8:46 PM To: Tamar Christina ; gcc-patches@gcc.gnu.org Cc: nd ; bonz...@gnu.org; nero...@gcc.gnu.org; aol...@gcc.gnu.org

RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-04 Thread Tamar Christina via Gcc-patches
08 AM > > To: Jeff Law ; gcc-patches@gcc.gnu.org > > Cc: nd ; bonz...@gnu.org; nero...@gcc.gnu.org; > > aol...@gcc.gnu.org; ralf.wildenh...@gmx.de > > Subject: RE: [PATCH 5/5] match.pd: Use splits in makefile and make > > configurable. > > > > > -Original Mess

RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-03 Thread Kyrylo Tkachov via Gcc-patches
> aol...@gcc.gnu.org; ralf.wildenh...@gmx.de > Subject: RE: [PATCH 5/5] match.pd: Use splits in makefile and make > configurable. > > > -Original Message- > > From: Jeff Law > > Sent: Sunday, April 30, 2023 8:46 PM > > To: Tamar Christina ; gcc-patches@gcc.

RE: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-05-02 Thread Tamar Christina via Gcc-patches
> -Original Message- > From: Jeff Law > Sent: Sunday, April 30, 2023 8:46 PM > To: Tamar Christina ; gcc-patches@gcc.gnu.org > Cc: nd ; bonz...@gnu.org; nero...@gcc.gnu.org; > aol...@gcc.gnu.org; ralf.wildenh...@gmx.de > Subject: Re: [PATCH 5/5] match.pd: Use splits

Re: [PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-04-30 Thread Jeff Law via Gcc-patches
On 4/28/23 04:44, Tamar Christina via Gcc-patches wrote: Hi All, This updates the build system to split up match.pd files into chunks of 10. This also introduces a new flag --with-matchpd-partitions which can be used to change the number of partitions. For the analysis of why 10 please look

[PATCH 5/5] match.pd: Use splits in makefile and make configurable.

2023-04-28 Thread Tamar Christina via Gcc-patches
Hi All, This updates the build system to split up match.pd files into chunks of 10. This also introduces a new flag --with-matchpd-partitions which can be used to change the number of partitions. For the analysis of why 10 please look at the previous patch in the series. Bootstrapped Regtested o