On April 27, 2016 5:27:32 PM GMT+02:00, Bernd Schmidt <bschm...@redhat.com>
wrote:
>On 03/28/2016 06:44 AM, Andi Kleen wrote:
>> From: Andi Kleen <a...@linux.intel.com>
>
>> @@ -313,6 +320,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
>> }
If neither perf nor data depend on $ext then it shouldn't be necessary to loop
them over $prof_ext, no?
thanks,
>
>We have this, and then...
>
>> @@ -400,6 +451,7 @@ proc profopt-execute { src } {
>> foreach ext $prof_ext {
>> profopt-target-cleanup $tmpdir $base $ext
>> }
>> + # XXX remove perf.data
>
>... this - does that need to look the same as the above?
>
>> + # Should check if create_gcov exists
>
>So maybe do that?
>
>
>Bernd
