[PATCH 3/5] Run profile feedback tests with autofdo

[Andi Kleen]

On Sat, May 21, 2016 at 10:55:21PM +0200, Bernhard Reutner-Fischer wrote:
> >@@ -313,6 +332,7 @@ proc profopt-execute { src } {
> > 	# valid, by running it after dg-additional-files-options.
> > 	foreach ext $prof_ext {
> > 	    profopt-target-cleanup $tmpdir $base $ext
> >+	    profopt-target-cleanup $tmpdir perf data
> > 	}
> 
> I think I've asked this before.. Why do you need to run cleanup of perf data in this loop (and not outside)?
> Also why the asymmetry ...

perf generates a backup file if a file already exists. 

> >@@ -399,7 +460,9 @@ proc profopt-execute { src } {
> > 	# Remove the profiling data files.
> > 	foreach ext $prof_ext {
> > 	    profopt-target-cleanup $tmpdir $base $ext
> >+	    profopt-target-cleanup $tmpdir perf data
> > 	}
> >+	file delete "perf.data"	

The extra one is not needed. I'll remove it.

Thanks
-Andi

-- 
ak@linux.intel.com -- Speaking for myself only.