[PATCH] D15462: [CMake] Add support for generating profdata for clang from training files

Sean Silva via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 15 15:52:35 PST 2015


silvas added a comment.

Thanks. This LGTM but I'd wait for Duncan or Justin to sign off on it; they're likely to have more higher-level thoughts.


================
Comment at: utils/perf-training/perf-helper.py:36
@@ +35,3 @@
+  cmd.extend(findProfrawFiles(args[2]))
+  subprocess.check_call(cmd, stdin=subprocess.PIPE, stdout=subprocess.PIPE)
+  return 0
----------------
I don't think you need the PIPE arguments (or maybe look into check_output if you want the output)


http://reviews.llvm.org/D15462





More information about the cfe-commits mailing list