[Bug driver/93645] Support -fuse-ld=/absolute/path/to/ld

[Bug driver/93645] Support -fuse-ld=/absolute/path/to/ld

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |10.0
   Last reconfirmed|                            |2020-03-11
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW
            Version|unknown                     |10.0
                 CC|                            |marxin at gcc dot gnu.org
           Keywords|                            |patch

[Bug driver/93645] Support -fuse-ld=/absolute/path/to/ld

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|10.0                        |10.2

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 10.1 has been released.

[Bug driver/93645] Support -fuse-ld=/absolute/path/to/ld

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|10.2                        |10.3

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
GCC 10.2 is released, adjusting target milestone.

[Bug driver/93645] Support Clang 12 --ld-path=

[i at maskray dot me]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #4 from Fangrui Song <i at maskray dot me> ---
https://sourceware.org/pipermail/gcc-patches/2020-July/550659.html [PATCH v3]
Add --ld-path= to specify an arbitrary executable as the linker


I changed the title to --ld-path because -fuse-ld=/absolute/path/to/ld is not a
good design. -fuse-ld= can mean the linker flavor (there can be option dispatch
on this option) & --ld-path can specify the path overriding -fuse-ld='s default
choice.

-f* options are usually about code generation or language features. --ld-path
does not belong to the category so -f is not very appropriate.

Clang 12 will have --ld-path.

[Bug driver/93645] Support Clang 12 --ld-path=

[i at maskray dot me]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #5 from Fangrui Song <i at maskray dot me> ---
Ping

[Bug driver/93645] Support Clang 12 --ld-path=

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|10.3                        |10.4

--- Comment #6 from Richard Biener <rguenth at gcc dot gnu.org> ---
GCC 10.3 is being released, retargeting bugs to GCC 10.4.

[Bug driver/93645] Support Clang 12 --ld-path=

[hjl.tools at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |freesoftware at logarithmus dot de
                   |                            |v

--- Comment #7 from H.J. Lu <hjl.tools at gmail dot com> ---
*** Bug 101215 has been marked as a duplicate of this bug. ***

[Bug driver/93645] Support Clang 12 --ld-path=

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |d.g.gorbachev at gmail dot com

--- Comment #8 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
*** Bug 56062 has been marked as a duplicate of this bug. ***

[Bug driver/93645] Support Clang 12 --ld-path=

[freesoftware at logarithmus dot dev]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #9 from Artur Sinila <freesoftware at logarithmus dot dev> ---
What's a blocker for this bug? What should be improved in the patch in order
for it to be accepted?

[Bug driver/93645] Support Clang 12 --ld-path=

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #10 from Martin Liška <marxin at gcc dot gnu.org> ---
I replied here:
https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573823.html

[Bug driver/93645] Support Clang 12 --ld-path=

[i at maskray dot me]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #11 from Fangrui Song <i at maskray dot me> ---
(In reply to Martin Liška from comment #10)
> I replied here:
> https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573823.html

There are people wanting to use mold
https://www.reddit.com/r/rust/comments/rhcnzt/mold_a_modern_linker_10_release/

"clang does support it but gcc: --ld-path patch has been declined by GCC
maintainers, instead they advise to use a workaround: create directory
<dirname>, then ln -s <path-to-mold> <dirname>/ld, and then pass -B<dirname>
(-B tells GCC to look for ld in specified location)."

:(

[Bug driver/93645] Support Clang 12 --ld-path=

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at gcc dot gnu.org      |marxin at gcc dot gnu.org

--- Comment #12 from Martin Liška <marxin at gcc dot gnu.org> ---
(In reply to Fangrui Song from comment #11)
> (In reply to Martin Liška from comment #10)
> > I replied here:
> > https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573823.html
> 
> There are people wanting to use mold
> https://www.reddit.com/r/rust/comments/rhcnzt/
> mold_a_modern_linker_10_release/

I agree that's unfortunate. Note I'm having a patch that adds -fuse-ld=mold:
https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=759cdbb29dbe8fc80ba5c1f113a015cafe9eb69c

I can try suggesting that to the community for GCC 12 (and maybe backport
that).
Are you interested?

Note the linker is very interesting, but it lacks LTO support..

[Bug driver/93645] Support Clang 12 --ld-path=

[i at maskray dot me]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #13 from Fangrui Song <i at maskray dot me> ---
(In reply to Martin Liška from comment #12)
> (In reply to Fangrui Song from comment #11)
> > (In reply to Martin Liška from comment #10)
> > > I replied here:
> > > https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573823.html
> > 
> > There are people wanting to use mold
> > https://www.reddit.com/r/rust/comments/rhcnzt/
> > mold_a_modern_linker_10_release/
> 
> I agree that's unfortunate. Note I'm having a patch that adds -fuse-ld=mold:
> https://gcc.gnu.org/git/?p=gcc.git;a=commit;
> h=759cdbb29dbe8fc80ba5c1f113a015cafe9eb69c
> 
> I can try suggesting that to the community for GCC 12 (and maybe backport
> that).
> Are you interested?

I think it may be useful to simply allow -fuse-ld=word (`word` cannot include a
separator).

If that may be troublesome, having -fuse-ld=mold in GCC 12 is still nice.

--ld-path is occasionally useful, but I can accept that GCC declines it.

> Note the linker is very interesting, but it lacks LTO support..

Right...

[Bug driver/93645] Support Clang 12 --ld-path=

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org

--- Comment #14 from Martin Liška <marxin at gcc dot gnu.org> ---
> 
> I think it may be useful to simply allow -fuse-ld=word (`word` cannot
> include a separator).

Sure, but Jakub had some concerns:
https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573833.html

> 
> If that may be troublesome, having -fuse-ld=mold in GCC 12 is still nice.
> 

I've just done that:
https://gcc.gnu.org/pipermail/gcc-patches/2021-December/587426.html

[Bug driver/93645] Support Clang 12 --ld-path=

[i at maskray dot me]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #15 from Fangrui Song <i at maskray dot me> ---
-- is definitely rare, but not non-existent.
In GCC, there is {-,--}specs.
In Clang, there are --cuda-path, --ptxas-path, --hip-path, --classpath, etc.
(In reply to Martin Liška from comment #14)
> > 
> > I think it may be useful to simply allow -fuse-ld=word (`word` cannot
> > include a separator).
> 
> Sure, but Jakub had some concerns:
> https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573833.html

I do not see an objection to -fuse-ld=word.

For --ld-path, -- is definitely rare, but not non-existent.
In GCC, there are {-,--}specs --sysroot.
In Clang, there are --cuda-path, --ptxas-path, --hip-path, --classpath, etc.

-fuse-ld= users mostly care about whether another linker can build their
programs, not whether the option can bootstrap GCC. I actually think ld.lld is
quite sufficient in bootstrapping GCC but if there are edge-case extensions
which not supported, ld.lld developers may not want to the project with more
obscure options...

> > 
> > If that may be troublesome, having -fuse-ld=mold in GCC 12 is still nice.
> > 
> 
> I've just done that:
> https://gcc.gnu.org/pipermail/gcc-patches/2021-December/587426.html

Thanks

[Bug driver/93645] Support Clang 12 --ld-path=

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

--- Comment #16 from Martin Liška <marxin at gcc dot gnu.org> ---
Note -fuse-ld=mold was added in: g:ad964f7eaef9c03ce68a01cfdd7fde9d56524868

[Bug driver/93645] Support Clang 12 --ld-path=

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|10.4                        |10.5

--- Comment #17 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 10.4 is being released, retargeting bugs to GCC 10.5.

[Bug driver/93645] Support Clang 12 --ld-path=

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93645

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|10.5                        |---