Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2021-06-28 Thread Jakub Jelinek via Gcc-patches
On Mon, Jun 28, 2021 at 04:41:06PM +0300, Artur Sinila wrote: > See https://reviews.llvm.org/D83015. Speaking about -B option, AFAIK it > will make gcc to search for collect2, cc and other tools in specified > directory, so apart from creating a symlink to mold, I'll also need to > create symlinks

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2021-06-28 Thread Jakub Jelinek via Gcc-patches
On Mon, Jun 28, 2021 at 03:26:21PM +0200, Martin Liška wrote: > > There are 2 options: > > 1. Add mold to -fuse-ld option > > 2. Implement --ld-path > > > > The 2nd option is much more future-proof: you won't need to add new > > -fuse-ld variant each time new linker comes up. To provide some > > c

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2021-06-28 Thread Artur Sinila via Gcc-patches
On Mon, 2021-06-28 at 15:26 +0200, Martin Liška wrote: > On 6/28/21 2:24 PM, Artur Sinila wrote: > > On Mon, 2021-06-28 at 14:08 +0200, Martin Liška wrote: > > > On 6/26/21 4:44 PM, Artur Sinila wrote: > > > > Not so gentle ping :) > > > > What should happen in order for this patch to be accepted?

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2021-06-28 Thread Martin Liška
On 6/28/21 2:24 PM, Artur Sinila wrote: On Mon, 2021-06-28 at 14:08 +0200, Martin Liška wrote: On 6/26/21 4:44 PM, Artur Sinila wrote: Not so gentle ping :) What should happen in order for this patch to be accepted? Hello. We came up to conclusion that one can use the currently supported op

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2021-06-28 Thread Artur Sinila via Gcc-patches
On Mon, 2021-06-28 at 14:08 +0200, Martin Liška wrote: > On 6/26/21 4:44 PM, Artur Sinila wrote: > > Not so gentle ping :) > > What should happen in order for this patch to be accepted? > > > > Hello. > > We came up to conclusion that one can use the currently supported > option > -fuse-ld={bfd,

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2021-06-28 Thread Martin Liška
On 6/26/21 4:44 PM, Artur Sinila wrote: Not so gentle ping :) What should happen in order for this patch to be accepted? Hello. We came up to conclusion that one can use the currently supported option -fuse-ld={bfd,gold,lld} with -B that can point to an arbitrary path the such linker is expec

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2021-06-26 Thread Artur Sinila via Gcc-patches
Not so gentle ping :) What should happen in order for this patch to be accepted? signature.asc Description: This is a digitally signed message part

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2020-04-06 Thread Martin Liška
On 4/6/20 12:32 AM, Fangrui Song wrote: On 2020-03-11, Martin Liška wrote: On 2/10/20 1:02 AM, Fangrui Song via gcc-patches wrote: Hello. Thank you for the patch. You haven't received a review because we are right now in stage4 of the development cycle: https://gcc.gnu.org/develop.html#stage4

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2020-04-05 Thread Fangrui Song via Gcc-patches
On 2020-03-11, Martin Liška wrote: On 2/10/20 1:02 AM, Fangrui Song via gcc-patches wrote: Hello. Thank you for the patch. You haven't received a review because we are right now in stage4 of the development cycle: https://gcc.gnu.org/develop.html#stage4 Thanks for the review! According to htt

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2020-03-11 Thread Martin Liška
On 2/10/20 1:02 AM, Fangrui Song via gcc-patches wrote: Hello. Thank you for the patch. You haven't received a review because we are right now in stage4 of the development cycle: https://gcc.gnu.org/develop.html#stage4 Anyway, I'm going to provide a review (even though I'm not maintainer of tha

[PING^3][PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2020-03-10 Thread Fangrui Song via Gcc-patches
On 2020-02-24, Fangrui Song wrote: On 2020-02-13, Fangrui Song wrote: On 2020-02-09, Fangrui Song wrote: PR driver/93645 * common.opt (-fuse-ld=): Delete -fuse-ld=[bfd|gold|lld]. Add -fuse-ld=. * opts.c (common_handle_option): Handle OPT_fuse_ld_. * gcc.c (driver

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2020-02-24 Thread Fangrui Song via gcc-patches
On 2020-02-13, Fangrui Song wrote: On 2020-02-09, Fangrui Song wrote: PR driver/93645 * common.opt (-fuse-ld=): Delete -fuse-ld=[bfd|gold|lld]. Add -fuse-ld=. * opts.c (common_handle_option): Handle OPT_fuse_ld_. * gcc.c (driver_handle_option): Likewise. *

Re: [PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2020-02-13 Thread Fangrui Song via gcc-patches
On 2020-02-09, Fangrui Song wrote: PR driver/93645 * common.opt (-fuse-ld=): Delete -fuse-ld=[bfd|gold|lld]. Add -fuse-ld=. * opts.c (common_handle_option): Handle OPT_fuse_ld_. * gcc.c (driver_handle_option): Likewise. * collect2.c (main): Likewise. --- gc

[PATCH] Generalize -fuse-ld= to support absolute path or arbitrary ld.linker

2020-02-09 Thread Fangrui Song via gcc-patches
PR driver/93645 * common.opt (-fuse-ld=): Delete -fuse-ld=[bfd|gold|lld]. Add -fuse-ld=. * opts.c (common_handle_option): Handle OPT_fuse_ld_. * gcc.c (driver_handle_option): Likewise. * collect2.c (main): Likewise. --- gcc/ChangeLog | 8 ++ gcc/c