This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix driver/39293
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, Ian Lance Taylor <iant at google dot com>, gcc-patches at gcc dot gnu dot org
- Date: Tue, 3 Mar 2009 20:29:31 +0100
- Subject: Re: [PATCH] Fix driver/39293
- References: <20090228010000.GA25597@hungry-tiger.westford.ibm.com> <m3k577v64z.fsf@google.com> <20090303192036.GA20604@hungry-tiger.westford.ibm.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Mar 03, 2009 at 02:20:36PM -0500, Michael Meissner wrote:
> > > This patch fixes bug 39293 by remembering the directory specified by an -o
> > > switch, adding a new spec code '%p' to emit this directory, including
> > > separator, and adding the %p's inside every %{save-temps} operation.
> >
> > This is a change in current behaviour. Are we sure that this change is
> > desirable? Is anybody concerned by the change?
>
> I must admit as the original author of -save-temps, many years ago (19 or 20?),
> I certainly did not anticipate people building objects in a different
> directory when I wrote it.
>
> I don't have a problem with adding a new switch, but I agree we already have
> too many weird switches.
Instead of adding a new switch, couldn't -save-temps just be extended to
accept an optional argument, the dirname + prefix for the temporary files?
-save-temps=/foo/bar/baz/abc
would create
/foo/bar/baz/abc.i
/foo/bar/baz/abc.s
etc.
Jakub