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

Justin Bogner via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 15 16:54:37 PST 2015


Sean Silva <chisophugis at gmail.com> writes:
> 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.

I'm fine with the concept and Sean's LGTM should cover the details :)

>
> ================
> 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