Re: [EXT] Re: [Driver] Add support for -fuse-ld=lld

2019-02-04 Thread Romain Geissler
On Fri, 25 Jan 2019, Jonathan Wakely wrote: > This patch was committed to trunk (r265940), but HJ's review comments > were never addressed (and look correct to me): > https://clicktime.symantec.com/3QJw9yc7Scra74WXzR3yBDR6H2?u=https%3A%2F%2Fgcc.gnu.org%2Fml%2Fgcc-patches%2F2016-07%2Fmsg00146.html

Re: [EXT] Re: [Driver] Add support for -fuse-ld=lld

2019-01-29 Thread Jonathan Wakely
On 29/01/19 14:12 -0500, Marek Polacek wrote: On Tue, Jan 29, 2019 at 07:07:31PM +, Romain GEISSLER wrote: > Le 29 janv. 2019 à 20:03, Marek Polacek a écrit : > > On Fri, Jan 25, 2019 at 03:06:41PM +, Jonathan Wakely wrote: > > Indeed. Romain, are you going to submit a followup patch t

Re: [EXT] Re: [Driver] Add support for -fuse-ld=lld

2019-01-29 Thread Marek Polacek
On Tue, Jan 29, 2019 at 07:07:31PM +, Romain GEISSLER wrote: > > Le 29 janv. 2019 à 20:03, Marek Polacek a écrit : > > > > On Fri, Jan 25, 2019 at 03:06:41PM +, Jonathan Wakely wrote: > > > > Indeed. Romain, are you going to submit a followup patch to address this? > > > > Marek > > H

Re: [EXT] Re: [Driver] Add support for -fuse-ld=lld

2019-01-29 Thread Romain GEISSLER
> Le 29 janv. 2019 à 20:03, Marek Polacek a écrit : > > On Fri, Jan 25, 2019 at 03:06:41PM +, Jonathan Wakely wrote: > > Indeed. Romain, are you going to submit a followup patch to address this? > > Marek Hi, I submitted the original patch because it was not written by me but by Davide,

Re: [Driver] Add support for -fuse-ld=lld

2019-01-29 Thread Marek Polacek
On Fri, Jan 25, 2019 at 03:06:41PM +, Jonathan Wakely wrote: > On 20/10/18 12:18 +0200, Romain Geissler wrote: > > Hi, > > > > I would like to raise again the question of supporting -fuse-ld=ldd. A > > patch implementing it was already submitted in > > https://gcc.gnu.org/ml/gcc-patches/2016-0

Re: [Driver] Add support for -fuse-ld=lld

2019-01-28 Thread Alan Modra
On Sat, Jan 26, 2019 at 04:34:59PM -0600, Segher Boessenkool wrote: > On Thu, Jun 23, 2016 at 09:01:30PM -0700, Davide Italiano wrote: > > LLVM currently ships with a new ELF linker http://lld.llvm.org/. > > I experiment a lot with gcc and lld so it would be nice if > > -fuse-ld=lld is supported (c

Re: [Driver] Add support for -fuse-ld=lld

2019-01-26 Thread Segher Boessenkool
On Thu, Jun 23, 2016 at 09:01:30PM -0700, Davide Italiano wrote: > LLVM currently ships with a new ELF linker http://lld.llvm.org/. > I experiment a lot with gcc and lld so it would be nice if > -fuse-ld=lld is supported (considering the linker is now mature enough > to link large C/C++ application

Re: [Driver] Add support for -fuse-ld=lld

2019-01-25 Thread Jonathan Wakely
On 20/10/18 12:18 +0200, Romain Geissler wrote: Hi, I would like to raise again the question of supporting -fuse-ld=ldd. A patch implementing it was already submitted in https://gcc.gnu.org/ml/gcc-patches/2016-06/msg01722.html by Davide Italiano. This patch still applies correctly to current tru

Re: [Driver] Add support for -fuse-ld=lld

2018-11-08 Thread Jeff Law
> > Is there any other reason why -fuse-ld=ldd shall not be supported by gcc ? > > Cheers, > Romain > > From 323c23d79c91d7dcee2f29b9ced8c1c00703d346 Mon Sep 17 00:00:00 2001 > From: Davide Italiano > Date: Thu, 23 Jun 2016 20:51:53 -0700 > Subject: [PA

Re: [EXT] Re: [Driver] Add support for -fuse-ld=lld

2018-11-08 Thread Romain Geissler
On Thu, 8 Nov 2018, Richard Biener wrote: The patch is OK. Thanks, Richard. Thanks. Can you please apply it as I don't have any commit rights ? The patch can be found in https://gcc.gnu.org/ml/gcc-patches/2018-10/msg01240.html Here is a valid gcc/ChangeLog correctly formated where I am ref

Re: [EXT] Re: [Driver] Add support for -fuse-ld=lld

2018-11-08 Thread Richard Biener
On Thu, Nov 8, 2018 at 12:00 PM Romain Geissler wrote: > > On Tue, 6 Nov 2018, H.J. Lu wrote: > > > On Sat, Oct 20, 2018 at 3:19 AM Romain Geissler > > wrote: > > > > > > I would like to raise again the question of supporting -fuse-ld=lld. > > > > > > > LGTM. But I can't approve it. > > > > -- >

Re: [EXT] Re: [Driver] Add support for -fuse-ld=lld

2018-11-08 Thread Romain Geissler
On Tue, 6 Nov 2018, H.J. Lu wrote: > On Sat, Oct 20, 2018 at 3:19 AM Romain Geissler > wrote: > > > > I would like to raise again the question of supporting -fuse-ld=lld. > > > > LGTM. But I can't approve it. > > -- > H.J. Hi, Is there any gcc reviewer that could comment whether or not this sm

Re: [Driver] Add support for -fuse-ld=lld

2018-11-06 Thread H.J. Lu
t; if yes, why ? > > Is there any other reason why -fuse-ld=ldd shall not be supported by gcc ? > > Cheers, > Romain > > From 323c23d79c91d7dcee2f29b9ced8c1c00703d346 Mon Sep 17 00:00:00 2001 > From: Davide Italiano > Date: Thu, 23 Jun 2016 20:51:53 -0700 > Subject:

Re: [Driver] Add support for -fuse-ld=lld

2018-11-06 Thread Romain GEISSLER
? >> >> Cheers, >> Romain >> >> From 323c23d79c91d7dcee2f29b9ced8c1c00703d346 Mon Sep 17 00:00:00 2001 >> From: Davide Italiano >> Date: Thu, 23 Jun 2016 20:51:53 -0700 >> Subject: [PATCH] Driver: Add support for -fuse-ld=lld. >> >

Re: [Driver] Add support for -fuse-ld=lld

2018-10-27 Thread Romain GEISSLER
oday, and > if yes, why ? > > Is there any other reason why -fuse-ld=ldd shall not be supported by gcc ? > > Cheers, > Romain > > From 323c23d79c91d7dcee2f29b9ced8c1c00703d346 Mon Sep 17 00:00:00 2001 > From: Davide Italiano > Date: Thu, 23 Jun 2016 20:51:53 -0700 &

[Driver] Add support for -fuse-ld=lld

2018-10-20 Thread Romain Geissler
Mon Sep 17 00:00:00 2001 From: Davide Italiano Date: Thu, 23 Jun 2016 20:51:53 -0700 Subject: [PATCH] Driver: Add support for -fuse-ld=lld. * collect2.c (main): Support -fuse-ld=lld. * common.opt: Add fuse-ld=lld * doc/invoke.texi: Document -fuse-ld=lld * opts.c: Ignore -fuse-ld=lld ---

Re: [Driver] Add support for -fuse-ld=lld

2016-07-06 Thread Trevor Saunders
On Mon, Jul 04, 2016 at 09:36:52PM +0200, Markus Trippelsdorf wrote: > On 2016.07.04 at 10:08 -0700, H.J. Lu wrote: > > On Sun, Jul 3, 2016 at 9:38 PM, Davide Italiano > > wrote: > > > On Thu, Jun 23, 2016 at 9:11 PM, Davide Italiano > > > wrote: > > >> + HJ who wrote the code for the option or

Re: [Driver] Add support for -fuse-ld=lld

2016-07-04 Thread Mike Stump
On Jul 4, 2016, at 12:36 PM, Markus Trippelsdorf wrote: > > On 2016.07.04 at 10:08 -0700, H.J. Lu wrote: >> On Sun, Jul 3, 2016 at 9:38 PM, Davide Italiano >> wrote: >>> On Thu, Jun 23, 2016 at 9:11 PM, Davide Italiano >>> wrote: + HJ who wrote the code for the option originally. >

Re: [Driver] Add support for -fuse-ld=lld

2016-07-04 Thread Davide Italiano
recipe for target > 'tools/lld/ELF/CMakeFiles/lldELF.dir/OutputSections.cpp.o' failed > gmake[4]: *** [tools/lld/ELF/CMakeFiles/lldELF.dir/OutputSections.cpp.o] > Error 1 > > Can you fix it? >> -- >> Davide >> >>>> -- >>>> Dav

Re: [Driver] Add support for -fuse-ld=lld

2016-07-04 Thread Davide Italiano
On Mon, Jul 4, 2016 at 12:36 PM, Markus Trippelsdorf wrote: > On 2016.07.04 at 10:08 -0700, H.J. Lu wrote: >> On Sun, Jul 3, 2016 at 9:38 PM, Davide Italiano >> wrote: >> > On Thu, Jun 23, 2016 at 9:11 PM, Davide Italiano >> > wrote: >> >> + HJ who wrote the code for the option originally. >>

Re: [Driver] Add support for -fuse-ld=lld

2016-07-04 Thread Markus Trippelsdorf
On 2016.07.04 at 10:08 -0700, H.J. Lu wrote: > On Sun, Jul 3, 2016 at 9:38 PM, Davide Italiano wrote: > > On Thu, Jun 23, 2016 at 9:11 PM, Davide Italiano > > wrote: > >> + HJ who wrote the code for the option originally. > >> > >> On Thu, Jun 23, 2016 at 9:01 PM, Davide Italiano > >> wrote: >

Re: [Driver] Add support for -fuse-ld=lld

2016-07-04 Thread H.J. Lu
th > mentioning. > > Thanks, > > -- > Davide > > From 323c23d79c91d7dcee2f29b9ced8c1c00703d346 Mon Sep 17 00:00:00 2001 > From: Davide Italiano > Date: Thu, 23 Jun 2016 20:51:53 -0700 > Subject: [PATCH] Driver: Add support for -fuse-ld=lld. > > * collect2.

Re: [Driver] Add support for -fuse-ld=lld

2016-07-04 Thread H.J. Lu
mentioning. >>> >>> Thanks, >>> > > Hi, can anybody take a look? lld isn't compatible with GCC: https://llvm.org/bugs/show_bug.cgi?id=28414 > Thanks, > > -- > Davide > >>> -- >>> Davide >>> >>> From 323c2

Re: [Driver] Add support for -fuse-ld=lld

2016-07-04 Thread H.J. Lu
/ELF/CMakeFiles/lldELF.dir/build.make:302: recipe for target 'tools/lld/ELF/CMakeFiles/lldELF.dir/OutputSections.cpp.o' failed gmake[4]: *** [tools/lld/ELF/CMakeFiles/lldELF.dir/OutputSections.cpp.o] Error 1 Can you fix it? H.J. > -- > Davide > >>> -- >>>

Re: [Driver] Add support for -fuse-ld=lld

2016-07-03 Thread Davide Italiano
t; Davide >> >> From 323c23d79c91d7dcee2f29b9ced8c1c00703d346 Mon Sep 17 00:00:00 2001 >> From: Davide Italiano >> Date: Thu, 23 Jun 2016 20:51:53 -0700 >> Subject: [PATCH] Driver: Add support for -fuse-ld=lld. >> >> * collect2.c (main): Support -fuse-ld=lld. &

Re: [Driver] Add support for -fuse-ld=lld

2016-06-23 Thread Davide Italiano
utside of the scope of this patch, but I thought worth > mentioning. > > Thanks, > > -- > Davide > > From 323c23d79c91d7dcee2f29b9ced8c1c00703d346 Mon Sep 17 00:00:00 2001 > From: Davide Italiano > Date: Thu, 23 Jun 2016 20:51:53 -0700 > Subject: [PATCH] Driver: Add

[Driver] Add support for -fuse-ld=lld

2016-06-23 Thread Davide Italiano
:00 2001 From: Davide Italiano Date: Thu, 23 Jun 2016 20:51:53 -0700 Subject: [PATCH] Driver: Add support for -fuse-ld=lld. * collect2.c (main): Support -fuse-ld=lld. * common.opt: Add fuse-ld=lld * doc/invoke.texi: Document -fuse-ld=lld * opts.c: Ignore -fuse-ld=lld --- gcc/collect