Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Richard Biener via Gcc-patches
On Fri, Nov 20, 2020 at 12:30 AM Eric Botcazou wrote: > > > Successfully bootstrapped/regtested on x86_64-linux and i686-linux, > > including make install which looked problematic in PR97911. > > > > Ok for trunk? > > I cannot really approve, but this looks like a step in the right direction.

Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Eric Botcazou
> Successfully bootstrapped/regtested on x86_64-linux and i686-linux, > including make install which looked problematic in PR97911. > > Ok for trunk? I cannot really approve, but this looks like a step in the right direction. -- Eric Botcazou

Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Jakub Jelinek via Gcc-patches
On Thu, Nov 19, 2020 at 03:50:27PM +0100, Jakub Jelinek via Gcc-patches wrote: > So, I think the problem is that for make .PHONY targets are just > "rebuilt" always, so it is very much undesirable for the cc1plus$(exeext) > etc. dependencies to include .PHONY targets, but I was using > them -

Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Jakub Jelinek via Gcc-patches
On Thu, Nov 19, 2020 at 01:10:18PM +0100, Eric Botcazou wrote: > > Does it? > > Only with --enable-link-serialization, or without that too? > > It's the default, i.e. without --enable-link-serialization. > > > Will have a look. > > Thanks. So, I think the problem is that for make .PHONY

Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Martin Liška
On 11/19/20 1:10 PM, Eric Botcazou wrote: Does it? Only with --enable-link-serialization, or without that too? It's the default, i.e. without --enable-link-serialization. Will have a look. Thanks. I see the same, I don't use --enable-link-serialization. Just run: make -j16 && make -j16

Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Eric Botcazou
> Does it? > Only with --enable-link-serialization, or without that too? It's the default, i.e. without --enable-link-serialization. > Will have a look. Thanks. -- Eric Botcazou

Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Jakub Jelinek via Gcc-patches
On Thu, Nov 19, 2020 at 12:55:42PM +0100, Eric Botcazou wrote: > > * Make-lang.in (d.serial): New goal. > > (.PHONY): Add d.serial d.prev. > > (d21$(exeext)): Depend on d.prev. Call LINK_PROGRESS. > > This seems to cause the binaries to be always relinked, for example from the >

Re: [PATCH] configury: --enable-link-serialization support

2020-11-19 Thread Eric Botcazou
> 2020-11-18 Jakub Jelinek > > gcc/ > * configure.ac: Add $lang.prev rules, INDEX.$lang and SERIAL_LIST and > SERIAL_COUNT variables to Make-hooks. > (--enable-link-serialization): New configure option. > * Makefile.in (DO_LINK_SERIALIZATION, LINK_PROGRESS): New

Re: [PATCH] configury: --enable-link-serialization support

2020-11-18 Thread Joseph Myers
On Wed, 18 Nov 2020, Jakub Jelinek via Gcc-patches wrote: > Bootstrapped/regtested again last night on x86_64-linux and i686-linux, ok > for trunk? OK. -- Joseph S. Myers jos...@codesourcery.com

[PATCH] configury: --enable-link-serialization support

2020-11-18 Thread Jakub Jelinek via Gcc-patches
Hi! Reposting with self-contained description per Joseph's request: When performing LTO bootstraps, especially when using tmpfs for /tmp, one can run a machine to halt when using higher levels of parallelism and a large number of FEs, because there are too many concurrent LTO link commands