Re: Build broken due to "[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2"

2011-10-29 Thread Hans-Peter Nilsson
> From: Andi Kleen > Date: Sat, 29 Oct 2011 10:09:48 +0200 > Oops. Can you please confirm this patch fixes it? My autotester is still busy after your commit, but has passed the point of failure. Thanks for fixing. brgds, H-P

Re: Build broken due to "[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2"

2011-10-29 Thread Andi Kleen
On Sat, Oct 29, 2011 at 10:09:48AM +0200, Andi Kleen wrote: > > This broke cross to cris-elf and I guess many other targets > > with TOT binutils, as follows: > > > > mv -f Tlto-wrapper lto-wrapper > > Oops. Can you please confirm this patch fixes it? I committed the patch with ChangeLog as as

Re: Build broken due to "[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2"

2011-10-29 Thread Andi Kleen
> This broke cross to cris-elf and I guess many other targets > with TOT binutils, as follows: > > mv -f Tlto-wrapper lto-wrapper Oops. Can you please confirm this patch fixes it? -Andi diff --git a/gcc/gcc-ar.c b/gcc/gcc-ar.c index fc7e4a2..1e86d20 100644 --- a/gcc/gcc-ar.c +++ b/gcc/gcc-ar.c

Build broken due to "[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2"

2011-10-28 Thread Hans-Peter Nilsson
> From: Andi Kleen > Date: Fri, 21 Oct 2011 01:55:48 +0200 > 2011-10-19 Andi Kleen > > * Makefile.in (MOSTLYCLEANFILES): Add gcc-ar/nm/ranlib. > (native): Add gcc-ar, gcc-nm, gcc-ranlib. > (AR_LIBS, gcc-ar, gcc-ar.o, gcc-ranlib, gcc-ranlib.o, > gcc-nm, gcc-nm.o, gcc

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2

2011-10-28 Thread Richard Guenther
On Fri, Oct 21, 2011 at 1:55 AM, Andi Kleen wrote: > From: Andi Kleen > > Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most > convenient way to get this into existing Makefiles is using small > wrappers that pass the plugin. This matches how other compilers > (LLVM, icc) do thi

Re: PING [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2

2011-10-25 Thread Diego Novillo
On 11-10-25 01:23 , Andi Kleen wrote: Andi Kleen writes: From: Andi Kleen Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most convenient way to get this into existing Makefiles is using small wrappers that pass the plugin. This matches how other compilers (LLVM, icc) do this

PING [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2

2011-10-25 Thread Andi Kleen
Andi Kleen writes: > From: Andi Kleen > > Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most > convenient way to get this into existing Makefiles is using small > wrappers that pass the plugin. This matches how other compilers > (LLVM, icc) do this too. Ping. Could someone ple

[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2

2011-10-20 Thread Andi Kleen
From: Andi Kleen Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most convenient way to get this into existing Makefiles is using small wrappers that pass the plugin. This matches how other compilers (LLVM, icc) do this too. My previous attempt at using shell scripts for this htt

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Joseph S. Myers
On Thu, 20 Oct 2011, Andi Kleen wrote: > collect2 does not fully match what gcc.c does I think, so there's > already some divergence. collect2 is always called from within the gcc driver, so it can rely on environment variables set by the driver. As I understand it, these wrappers are not call

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Joseph S. Myers
On Thu, 20 Oct 2011, Andi Kleen wrote: > The logic gcc.c uses to find the files is very complicated. I didn't > try to replicate it 100% and left out some magic. I would be interested > if this simple method works for everyone or if more code needs > to be added. This only needs to support LTO sup

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Andi Kleen
On Thu, Oct 20, 2011 at 10:45:31AM +0200, Richard Guenther wrote: > > My previous attempt at using shell scripts for this > > http://gcc.gnu.org/ml/gcc-patches/2010-10/msg02471.html > > was not approved. Here's another attempt using wrappers written > > in C. It's only a single wrapper which just a

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 10:56 AM, Jan Hubicka wrote: >> On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen wrote: >> > From: Andi Kleen >> > >> > Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most >> > convenient way to get this into existing Makefiles is using small >> > wrappers tha

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Jan Hubicka
> On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen wrote: > > From: Andi Kleen > > > > Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most > > convenient way to get this into existing Makefiles is using small > > wrappers that pass the plugin. This matches how other compilers > > (LLV

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Richard Guenther
On Thu, Oct 20, 2011 at 9:24 AM, Andi Kleen wrote: > From: Andi Kleen > > Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most > convenient way to get this into existing Makefiles is using small > wrappers that pass the plugin. This matches how other compilers > (LLVM, icc) do thi

[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO

2011-10-20 Thread Andi Kleen
From: Andi Kleen Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most convenient way to get this into existing Makefiles is using small wrappers that pass the plugin. This matches how other compilers (LLVM, icc) do this too. My previous attempt at using shell scripts for this htt