[PATCH] D11725: [DependenceAnalysis] Ensure All Recurrences are Affine

Jingyue Wu via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 11 20:50:24 PDT 2015


jingyue resigned from this revision.
jingyue removed a reviewer: jingyue.
jingyue added a comment.

I don't see any obvious problem, but I don't have enough expertise on DependenceAnalysis to LGTM it unfortunately.


================
Comment at: lib/Analysis/DependenceAnalysis.cpp:3244
@@ -3243,1 +3243,3 @@
 
+/// Ensure all recurrences in an expression are affine.
+static bool allAddRecsAffine(const SCEVAddRecExpr *S) {
----------------
Nit: the comment is incorrect. This function only checks if all recurrences are affine instead of ensuring that. 


http://reviews.llvm.org/D11725





More information about the llvm-commits mailing list