[PATCH] D17080: [LAA] Allow more run-time alias checks by coercing pointer expressions to AddRecExprs

Dorit Nuzman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 00:42:01 PST 2017


dorit added inline comments.


================
Comment at: lib/Analysis/LoopAccessAnalysis.cpp:475
+                            Loop *TheLoop, unsigned &RunningDepId,
+                            unsigned ASId, bool ShouldCheckStride, bool Force);
+
----------------
I just have a small suggestion, to maybe change "Force" to "Assume", just because "Force" here has the same effect as "Assume" in the getPtrStride API, namely to allow adding new runtime tests. (Right?). But if you prefer Force that's fine with me too.


https://reviews.llvm.org/D17080





More information about the llvm-commits mailing list