[PATCH] D40723: [MemCpyOpt] Perform call slot optimizations through GEPs
Hal Finkel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 15 05:41:22 PST 2017
hfinkel added a comment.
In https://reviews.llvm.org/D40723#956485, @dotdash wrote:
> In https://reviews.llvm.org/D40723#955555, @rnk wrote:
>
> > I think you could split this up and write more targetted tests. The sret change needs a test and is probably separable.
>
>
> I didn't realize that phabricator only displays the list of commits, but provides no access to the individual diffs.
>
> So I should submit the fixes incrementally in multiple, sequential review requests, right?
Yes. Please post multiple phabricator review requests.
https://reviews.llvm.org/D40723
More information about the llvm-commits
mailing list