[PATCH] D60336: [test-suite] litsupport/modules/perf.py: store profile filename in context.profilefile

[Phabricator via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL358085: [test-suite] litsupport/modules/perf.py: store profile filename in context. (authored by lebedevri, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D60336?vs=193945&id=194509#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60336/new/

https://reviews.llvm.org/D60336

Files:
  test-suite/trunk/litsupport/modules/perf.py


Index: test-suite/trunk/litsupport/modules/perf.py
===================================================================
--- test-suite/trunk/litsupport/modules/perf.py
+++ test-suite/trunk/litsupport/modules/perf.py
@@ -6,12 +6,13 @@
 
 
 def _mutateCommandLine(context, commandline):
-    profilefile = context.tmpBase + ".perf_data"
+    context.profilefile = context.tmpBase + ".perf_data"
+    # Storing profile file in context allows other modules to be aware of it.
     cmd = shellcommand.parse(commandline)
     cmd.wrap('perf', [
         'record',
         '-e', context.config.perf_profile_events,
-        '-o', profilefile,
+        '-o', context.profilefile,
         '--'
     ])
     if cmd.stdout is None:
@@ -33,4 +34,4 @@
     script = testplan.mutateScript(context, script, _mutateCommandLine)
     plan.profilescript += script
     plan.metric_collectors.append(
-        lambda context: {'profile': context.tmpBase + '.perf_data'})
+        lambda context: {'profile': context.profilefile})


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60336.194509.patch
Type: text/x-patch
Size: 1012 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190410/315f94d3/attachment.bin>