Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-05-18 Thread Ludovic Courtès
Ramana Radhakrishnan skribis: > On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès wrote: >> As discussed at <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65711>. >> >> Patch is for both 4.8 and 4.9 (possibly 5.1 too, I haven’t checked.) >> > > OK for

[PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-04-23 Thread Ludovic Courtès
As discussed at <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65711>. Patch is for both 4.8 and 4.9 (possibly 5.1 too, I haven’t checked.) Tested on armhf-linux-gnu (armv7.) gcc/ 2015-04-23 Ludovic Courtès PR 65711 * config/arm/linux-elf.h (LINUX_TARGET_LINK_SPEC)

[PATCH COMMITTED] MAINTAINERS (Write After Approval): Add myself.

2015-06-23 Thread Ludovic Courtès
FYI. Index: ChangeLog === --- ChangeLog (revision 224824) +++ ChangeLog (revision 224825) @@ -1,3 +1,7 @@ +2015-06-23 Ludovic Courtès + + * MAINTAINERS (Write After Approval): Add myself. + 2015-06-22 Andreas Tobler

Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-06-23 Thread Ludovic Courtès
Ramana Radhakrishnan skribis: > On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès wrote: >> As discussed at <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65711>. >> >> Patch is for both 4.8 and 4.9 (possibly 5.1 too, I haven’t checked.) >> > > OK for

Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-06-23 Thread Ludovic Courtès
Richard Biener skribis: > On Mon, May 18, 2015 at 9:09 PM, Ludovic Courtès wrote: >> Ramana Radhakrishnan skribis: >> >>> On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès wrote: >>>> As discussed at <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65711>

Re: [PATCH][ARM] PR/65711: Don't pass '-dynamic-linker' when '-shared' is used

2015-06-23 Thread Ludovic Courtès
Jakub Jelinek skribis: > On Tue, Jun 23, 2015 at 11:30:25AM +0200, Ludovic Courtès wrote: >> Ramana Radhakrishnan skribis: >> >> > On Thu, Apr 23, 2015 at 9:29 AM, Ludovic Courtès wrote: >> >> As discussed at <https://gcc.gnu.org/bugzilla/show_bug.cgi?i

[PATCH] Pass LDFLAGS when building libgcc

2013-02-26 Thread Ludovic Courtès
Hello, In FLAGS_TO_PASS, libgcc/Makefile.in refers to $(LDFLAGS), but that variable is never set, so empty in practice. I believe This patches fixes that. Thanks, Ludo’. --- gcc-4.7.2/libgcc/Makefile.in~ 2012-09-05 14:19:47.0 +0200 +++ gcc-4.7.2/libgcc/Makefile.in 2013-02-26 20:26:15.00

Re: [PATCH] Pass LDFLAGS when building libgcc

2013-02-26 Thread Ludovic Courtès
Ian Lance Taylor skribis: > On Tue, Feb 26, 2013 at 11:28 AM, Ludovic Courtès wrote: >> >> In FLAGS_TO_PASS, libgcc/Makefile.in refers to $(LDFLAGS), but that >> variable is never set, so empty in practice. I believe This patches >> fixes that. > > This is O

Re: [PATCH] Pass LDFLAGS when building libgcc

2013-02-27 Thread Ludovic Courtès
Dave Korn skribis: > On 26/02/2013 22:07, Ludovic Courtès wrote: > >> * Makefile.in (LDFLAGS): New variable. > > "Import from automake" might be more informative. Automake has nothing to do with that. Ludo’.

[PATCH] Don’t insert white space in ‘orig_option_with_args_text’ for OPT_l

2012-03-09 Thread Ludovic Courtès
-l -L/nix/store/blsdhiik2lk4zmz3hbzf77g6hcrq7ckx-gfortran-wrapper-4.6.3/bin […] leading to an error like: ld: cannot find -l-L/nix/store/blsdhiik2lk4zmz3hbzf77g6hcrq7ckx-gfortran-wrapper-4.6.3/bin Ouch!) Thanks, Ludo’. 2012-03-09 Ludovic Courtès * gcc/opts-common.c (generate_opti

Re: [PATCH] Don't insert white space in 'orig_option_with_args_text' for OPT_l

2012-03-12 Thread Ludovic Courtès
Hi, Thanks for the review. "Joseph S. Myers" skribis: > On Fri, 9 Mar 2012, Ludovic Courtès wrote: > >> * gcc/opts-common.c (generate_option): Don't insert white space in >> `canonical_option' when OPT_INDEX is OPT_l. > > No, opts-common.c

Re: [PATCH] Don't insert white space in 'orig_option_with_args_text' for OPT_l

2012-03-13 Thread Ludovic Courtès
Hi, (Cc: Paul Brook.) "Joseph S. Myers" skribis: > On Mon, 12 Mar 2012, Ludovic Courtès wrote: > >> The patch below solves the problem in a gfortran-specific way. WDYT? > > I think that's the right approach for this issue. The previous patch was produced wi

[PATCH, ping #1] Don't insert white space in 'orig_option_with_args_text' for OPT_l

2012-03-29 Thread Ludovic Courtès
Ping. ludovic.cour...@inria.fr (Ludovic Courtès) skribis: > "Joseph S. Myers" skribis: > >> On Mon, 12 Mar 2012, Ludovic Courtès wrote: >> >>> The patch below solves the problem in a gfortran-specific way. WDYT? >> >> I think that's the rig