[PATCH] D22792: VecClone Pass

Matt via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 21 17:08:25 PST 2017


mmasten added a comment.

Thanks for the comments, Hal. Just to clarify your point #2, I think what you're saying is that we should start from a common parameter representation; i.e., parameters should be loaded/stored through memory. Please correct me if I'm wrong. I certainly think this would be a great way to reduce the complexity of the algorithm. The remainder of items in your list should already be covered, but some tweaking may be involved.


https://reviews.llvm.org/D22792





More information about the llvm-commits mailing list