[PATCH] D5226: [Polly] Support memory intrinsics

[Johannes Doerfert via llvm-commits]

jdoerfert marked 5 inline comments as done.

================
Comment at: lib/Analysis/ScopInfo.cpp:2576
@@ -2575,3 +2615,1 @@
       AG.push_back(PtrToAcc[PR.getValue()]);
-    assert(AG.size() > 1 &&
-           "Alias groups should contain at least two accesses");
----------------
etherzhhb wrote:
> why we relax this assertion?
Because the alias sets in the alias set tracker look differen when we also add memory intrinsics. This does not change the semantics of the algorithm though, it just refrlects that the alias groups can be small now such that we do not care.


http://reviews.llvm.org/D5226