PATCH: Add -plugin-save-temps

[Alan Modra]

On Mon, May 16, 2011 at 08:41:09AM -0700, H.J. Lu wrote:
> On Mon, May 16, 2011 at 8:20 AM, Alan Modra <amodra@gmail.com> wrote:
> > On Mon, May 16, 2011 at 07:35:35AM -0700, H.J. Lu wrote:
> >> On Mon, May 16, 2011 at 7:21 AM, Alan Modra <amodra@gmail.com> wrote:
> >> > On Sun, May 15, 2011 at 02:15:03PM -0700, H.J. Lu wrote:
> >> >> Â  Â  Â  (plugin_call_cleanup): Don't call plugin cleanup_handler if
> >> >> Â  Â  Â  plugin_save_temps is true.
> >> >
> >> > Isn't this really just covering for gcc -save-temps bugs?
> >> >
> >>
> >> It isn't straight forward since LTO plugin is invoked by linker,
> >> not gcc driver. Ultimately LTO cleanup is controlled by linker.
> >
> > Yes, but lto-wrapper modifies its cleanup according to whether
> > COLLECT_GCC_OPTIONS contains -save-temps. Â Why can't lto-plugin do the
> > same?
> 
> 1. It doesn't solve the problem for existing GCC.
> 2. To debug linker LTO bugs, we don't need other GCC
> intermediate files.

Both true, but it's easy to get the object files if you are in the
middle of a debug session.
a) I'd rather not have another linker switch to work around what looks
like a gcc bug to me, and
b) The plugin cleanup handler might have other duties besides deleting
temporary files in some future version.

-- 
Alan Modra
Australia Development Lab, IBM