[PATCH] D37355: Add CalledValuePropagation pass

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 13:38:02 PDT 2017


mssimpso added a comment.

In https://reviews.llvm.org/D37355#892441, @davide wrote:

> Yes, I'm fine with the dependent patches.
>  As this is a large-ish change (and involves a fair amount of refactoring), if I was you, I'd send a mail to llvm-dev to avoid surprises explaining what's your plan (so that if there are people relying on bits being in a particular position for their out-of-tree passes/analyses they at least have an heads-up of what's going on)


Yes, will do. Thanks, Davide. Danny has some remaining comments on the other patches that I'll take care of in the meantime.


https://reviews.llvm.org/D37355





More information about the llvm-commits mailing list