[PATCH] D45066: [Polly] [ScopInfo] Remove bail out condition in buildMinMaxAccess()

SAHIL GIRISH YERAWAR via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 16 00:19:23 PDT 2018


cs15btech11044 added a comment.





================
Comment at: lib/Analysis/ScopInfo.cpp:2312
 
-  Set = Set.remove_divs();
+  polly::simplify(Set);
+
----------------
Meinersbur wrote:
> Calling `remove_divs` before `simplify` could still be useful. It should not affect the ranges we need to check for aliasing.
Okay, I understand. This will be added in the next patch.


https://reviews.llvm.org/D45066





More information about the llvm-commits mailing list