Re: PING^2: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2013-01-07 Thread Joseph S. Myers
On Wed, 19 Dec 2012, H.J. Lu wrote: > +#ifdef CROSS_DIRECTORY_STRUCTURE > + = concat(target_machine, "-", ld_suffixes[i], NULL); Missing space before '('. The patch is OK with that fixed. -- Joseph S. Myers jos...@codesourcery.com

PING^3: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-12-31 Thread H.J. Lu
On Wed, Dec 19, 2012 at 2:20 PM, H.J. Lu wrote: > On Wed, Dec 19, 2012 at 1:13 PM, Joseph S. Myers > wrote: >> On Thu, 13 Dec 2012, H.J. Lu wrote: >> >>> Hi Joseph, >>> >>> Can you review this? >> >> I'm still confused as to what's supposed to be reviewed and whether the >> issues raised in the p

Re: PING^2: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-12-19 Thread H.J. Lu
On Wed, Dec 19, 2012 at 1:13 PM, Joseph S. Myers wrote: > On Thu, 13 Dec 2012, H.J. Lu wrote: > >> Hi Joseph, >> >> Can you review this? > > I'm still confused as to what's supposed to be reviewed and whether the > issues raised in the previous discussions have been properly addressed. > Looking a

Re: PING^2: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-12-19 Thread Joseph S. Myers
On Thu, 13 Dec 2012, H.J. Lu wrote: > Hi Joseph, > > Can you review this? I'm still confused as to what's supposed to be reviewed and whether the issues raised in the previous discussions have been properly addressed. Looking at , I do

PING^2: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-12-13 Thread H.J. Lu
On Fri, Dec 7, 2012 at 5:15 AM, H.J. Lu wrote: > On Wed, Nov 28, 2012 at 8:18 PM, H.J. Lu wrote: >> >> Binutils supports 2 linkers, ld.gold and ld.bfd. One of them is >> configured as the default linker, ld, which is used by GCC. Sometimes, >> we want to use the alternate linker with GCC at run

PING: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-12-07 Thread H.J. Lu
On Wed, Nov 28, 2012 at 8:18 PM, H.J. Lu wrote: > > Binutils supports 2 linkers, ld.gold and ld.bfd. One of them is > configured as the default linker, ld, which is used by GCC. Sometimes, > we want to use the alternate linker with GCC at run-time. This patch > adds -fuse-ld=bfd and -fuse-ld=go

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread H.J. Lu
On Thu, Nov 29, 2012 at 12:43 AM, Richard Biener wrote: > On Thu, Nov 29, 2012 at 5:18 AM, H.J. Lu wrote: >> From: "H.J. Lu" >> To: gcc-patches@gcc.gnu.org >> Cc: Joseph Myers , Paolo Bonzini >> Bcc: >> Subject: [PATCH] Support -fuse-ld=

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Matthias Klose
Am 29.11.2012 10:10, schrieb Markus Trippelsdorf: > Additionally, what is the rationale for this patch? IOW why isn't the > following > enough? > > x4 ~ # ln -f /usr/x86_64-pc-linux-gnu/binutils-bin/git/ld.gold > /usr/x86_64-pc-linux-gnu/binutils-bin/git/ld > x4 ~ # ld -v > GNU gold (GNU Binuti

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Matthias Klose
Am 29.11.2012 09:43, schrieb Richard Biener: > Do we need to consider GNU ld and gold coming from different binutils versions > and thus do we need two sets of linker feature tests at configure > time? both GNU ld and gold are built from the same sources. So it is likely that they come from the sa

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread H.J. Lu
Bonzini >> > >> > Bcc: >> > Subject: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold >> > Reply-To: >> > >> > Hi, >> > >> > Binutils supports 2 linkers, ld.gold and ld.bfd. One of them is >> > configured as the d

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Markus Trippelsdorf
On 2012.11.29 at 09:43 +0100, Richard Biener wrote: > On Thu, Nov 29, 2012 at 5:18 AM, H.J. Lu wrote: > > From: "H.J. Lu" > > To: gcc-patches@gcc.gnu.org > > Cc: Joseph Myers , Paolo Bonzini > > Bcc: > > Subject: [PATCH] Support -fuse-ld=bfd

Re: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-29 Thread Richard Biener
On Thu, Nov 29, 2012 at 5:18 AM, H.J. Lu wrote: > From: "H.J. Lu" > To: gcc-patches@gcc.gnu.org > Cc: Joseph Myers , Paolo Bonzini > Bcc: > Subject: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold > Reply-To: > > Hi, > > Binutils supports 2 link

[PATCH] Support -fuse-ld=bfd and -fuse-ld=gold

2012-11-28 Thread H.J. Lu
From: "H.J. Lu" To: gcc-patches@gcc.gnu.org Cc: Joseph Myers , Paolo Bonzini Bcc: Subject: [PATCH] Support -fuse-ld=bfd and -fuse-ld=gold Reply-To: Hi, Binutils supports 2 linkers, ld.gold and ld.bfd. One of them is configured as the default linker, ld, which is used by GCC. Som