[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