This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, PR65126, testsuite] Add additional_sources_used


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<tom@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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]