[PATCH] Fix driver/39293

Jakub Jelinek jakub@redhat.com
Tue Mar 3 19:29:00 GMT 2009


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



More information about the Gcc-patches mailing list