Re: [PATCH, PR65126, testsuite] Add additional_sources_used

[Jeff Law]

On 02/23/15 00:23, Tom de Vries wrote:

Hi,

This patch fixes PR65126, which is about the fact that the cleanup procs
are not cleaning up files generated from additional_sources. This
happens because dg-additional-files-options empties additional_sources
before the cleanup procs can use it.

Examples of leftover files because of this PR are:
- vect-simd-clone-10a.c.126t.vect
- pr47218-1.o, pr47218-1.ii, pr47218-1.s

The patch introduces a variable additional_sources_used, which is set by
dg-additional-files-options before it empties additional_sources.  The
cleanup procs are modified to use additional_sources_used.

With this patch, after running test I no longer find the leftover files
in the test directories.

Build and reg-tested on x86_64. OK for stage4?

Thanks,
- Tom

0001-Add-additional_sources_used.patch


2015-02-22  Tom de Vries<t...@codesourcery.com>

        * lib/gcc-defs.exp (additional_sources_used): Add new var.
        (dg-additional-files-options): Copy additional_sources to
        additional_sources_used before emptying additional_sources.
        * lib/gcc-dg.exp (cleanup-coverage-files, cleanup-repo-files)
        (cleanup-stack-usage, cleanup-ada-spec, cleanup-dump)
        (cleanup-saved-temps): Use additional_sources_used instead of
        additional_sources.
        (dg-test): Reset additional_sources_used.

OK.

Jeff