[PATCH] D23256: Teach the analysis manager about inter-analysis dependencies.
Sean Silva via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 8 23:55:57 PDT 2016
silvas added inline comments.
================
Comment at: include/llvm/Analysis/AliasAnalysis.h:894
@@ -895,1 +893,3 @@
+ 4>
+ ResultGetters;
----------------
mehdi_amini wrote:
> Wasn't the point of the refactoring with typedef to avoid this kind of diff?
I didn't eliminate all cases, but I did catch most. This case had a diff hunk that included a proxy change (below) so I kept it in this diff.
(I had to sort through hundreds of diff hunks so I was literally doing y/n to `git checkout -p` in order to get through them in a reasonable time)
https://reviews.llvm.org/D23256
More information about the llvm-commits
mailing list