llvm.org GIT mirror llvm / e41f890
[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. git-svn-id: https://llvm.org/svn/llvm-project/llvm/branches/release_39@276337 91177308-0d34-0410-b5e6-96231b3b80d8 George Burgess IV 4 years ago
1 changed file(s) with 0 addition(s) and 1 deletion(s). Raw diff Collapse all Expand all
12241224 MemoryAccess *CacheAccess = DFI.getPath(N - 1);
12251225 doCacheInsert(CacheAccess, ModifyingAccess, Q, Loc);
12261226 }
1227 assert(Q.Visited.size() < 1000 && "Visited too much");
12291228 return {ModifyingAccess, Loc};
12301229 }