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
On 10/20/18 4:18 AM, 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 curre

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
On Sat, Oct 20, 2018 at 3:19 AM 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 corre

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

2018-11-06 Thread Romain GEISSLER
Ping^2 > Le 27 oct. 2018 à 11:33, Romain GEISSLER a > écrit : > > Ping > >> Le 20 oct. 2018 à 12:18, Romain Geissler a >> écrit : >> >> 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/

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

2018-10-27 Thread Romain GEISSLER
Ping > Le 20 oct. 2018 à 12:18, Romain Geissler a > écrit : > > 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 appl

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
On Mon, Jul 4, 2016 at 9:12 AM, 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 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
On Thu, Jun 23, 2016 at 9:01 PM, 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++ applications). > >

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

2016-07-04 Thread H.J. Lu
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: >>> LLVM currently ships with a new ELF linker http://lld.ll

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

2016-07-04 Thread H.J. Lu
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: >>> LLVM currently ships with a new ELF linker http://lld.ll

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

2016-07-03 Thread Davide Italiano
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: >> LLVM currently ships with a new ELF linker http://lld.llvm.org/. >> I experiment a lot with gcc and lld so it would be nic

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

2016-06-23 Thread Davide Italiano
+ HJ who wrote the code for the option originally. On Thu, Jun 23, 2016 at 9:01 PM, 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 mat