[PATCH] D22825: Add a counter-function insertion pass

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 26 13:55:18 PDT 2016


hfinkel added a comment.

In https://reviews.llvm.org/D22825#496546, @rjmccall wrote:

> This seems like the right approach to me; I won't review the pass parts, though.  Should we do this + the change to actually start setting the flags in one patch?


Do you prefer that? I generally prefer when people split the Clang and LLVM patches into separate reviews (especially when they're functionally separable). On other other hand, if we move everything to one git repository, single patches will be easier...


https://reviews.llvm.org/D22825





More information about the llvm-commits mailing list