Author: gbiv
Date: Thu Jul 21 16:09:24 2016
New Revision: 276337
URL: http://llvm.org/viewvc/llvm-project?rev=276337&view=rev
Log:
[MSSA] Fix PR28632 in the 3.9 branch.
The now-removed assertion was really more for initial debugging; it's
perfectly valid (albeit relatively rare) for `Q.Visited.size()` to be
greater than 1,000. A similar patch hasn't been applied to trunk
because the piece of code this assertion was a part of no longer exists
in trunk.
Fix okayed by Danny, merge to 3.9 okayed by Hans.
Modified:
llvm/branches/release_39/lib/Transforms/Utils/MemorySSA.cpp
Modified: llvm/branches/release_39/lib/Transforms/Utils/MemorySSA.cpp
URL:
http://llvm.org/viewvc/llvm-project/llvm/branches/release_39/lib/Transforms/Utils/MemorySSA.cpp?rev=276337&r1=276336&r2=276337&view=diff
==============================================================================
--- llvm/branches/release_39/lib/Transforms/Utils/MemorySSA.cpp (original)
+++ llvm/branches/release_39/lib/Transforms/Utils/MemorySSA.cpp Thu Jul 21
16:09:24 2016
@@ -1225,7 +1225,6 @@ MemoryAccessPair MemorySSA::CachingWalke
MemoryAccess *CacheAccess = DFI.getPath(N - 1);
doCacheInsert(CacheAccess, ModifyingAccess, Q, Loc);
}
- assert(Q.Visited.size() < 1000 && "Visited too much");
return {ModifyingAccess, Loc};
}
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
