On 11/13/19 11:49 PM, Jan Hubicka wrote:
Hi.Current lto-dump.o relies on some FE generated files as pre-requirement. That hover delays LTO linking of the lto-dump and so that I adjusted the dependency to LTO_OBJS which will work as well. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin gcc/lto/ChangeLog: 2019-11-11 Martin Liska <mli...@suse.cz> * Make-lang.in: Relax dependency of lto-dump.o to LTO_OBJS which will allow faster linking (mainly with LTO).Hi, thanks for looking into this. Unforutnately it seems that even after your patch lto-dump always links only after all other frontends are finished :(
You are right :) I've got a patch that removes the not needed (and blocking) dependency. I'm going to install the patch after proper testing. Martin
Honza
>From f1f48015c12600e03c7da710f0b07185fa0c2572 Mon Sep 17 00:00:00 2001 From: Martin Liska <mli...@suse.cz> Date: Thu, 14 Nov 2019 09:16:58 +0100 Subject: [PATCH] Remove wrong lto-dump: lto1 makefile dependency. gcc/lto/ChangeLog: 2019-11-14 Martin Liska <mli...@suse.cz> * Make-lang.in: Remove wrong dependency of LTO_DUMP_EXE on LTO_EXE. --- gcc/lto/Make-lang.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/lto/Make-lang.in b/gcc/lto/Make-lang.in index faee8899502..46df75ab59b 100644 --- a/gcc/lto/Make-lang.in +++ b/gcc/lto/Make-lang.in @@ -88,7 +88,7 @@ $(LTO_EXE): $(LTO_OBJS) $(BACKEND) $(LIBDEPS) +$(LLINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) -o $@ \ $(LTO_OBJS) $(BACKEND) $(BACKENDLIBS) $(LIBS) -$(LTO_DUMP_EXE): $(LTO_EXE) $(LTO_DUMP_OBJS) $(BACKEND) $(LIBDEPS) +$(LTO_DUMP_EXE): $(LTO_DUMP_OBJS) $(BACKEND) $(LIBDEPS) +$(LLINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) -o $@ \ $(LTO_DUMP_OBJS) $(BACKEND) $(BACKENDLIBS) $(LIBS) -- 2.24.0
