This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch][GCC] Document and fix -r (partial linking)
- From: Allan Sandfeld Jensen <linux at carewolf dot com>
- To: Joseph Myers <joseph at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sun, 26 Aug 2018 12:34:06 +0200
- Subject: Re: [Patch][GCC] Document and fix -r (partial linking)
- References: <5835862.YDeY9hZc7r@twilight> <170372807.gIeshq9i0a@twilight> <alpine.DEB.2.21.1808202237010.646@digraph.polyomino.org.uk>
On Dienstag, 21. August 2018 00:38:58 CEST Joseph Myers wrote:
> On Fri, 3 Aug 2018, Allan Sandfeld Jensen wrote:
> > > I think you're changing the wrong place for this. If you want -r to be
> > > usable with GCC without using -nostdlib (which is an interesting
> > > question), you actually need to change LINK_COMMAND_SPEC (also sometimes
> > > overridden for targets) to handle -r more like -nostdlib -nostartfiles.
> >
> > Okay, so like this?
>
> Could you confirm if this has passed a bootstrap and testsuite run, with
> no testsuite regressions compared to GCC without the patch applied? I
> think it looks mostly OK (modulo ChangeLog rewrites and a missing second
> space after '.' in the manual change) but I'd like to make sure it's
> passed the usual testing before preparing it for commit.
I didn't think of running the tests since it only affects command line
options, but it did bootstrap, and behave as expected for my specific usecase.
I will update and run the tests when I have time.