[PATCH] D149251: [SCEV] Don't expect BackedgeTakenCounts cache to contain an entry for loop in getBackedgeTakenInfo

Dmitry Makogon via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 28 05:57:53 PDT 2023


dmakogon updated this revision to Diff 517884.
dmakogon added a comment.

Reworked the change as discussed offline with Max. The recursive call could happen in code that updates some stats. Namely in BackedgeTakenInfo::getExact. But the only usage of its result was to compare it against SCEVCouldNotCompute. So I added a new API to BackedgeTakenInfo which returns whether it can compute the exact exit count for loop and used it instead of its computation.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149251/new/

https://reviews.llvm.org/D149251

Files:
  llvm/include/llvm/Analysis/ScalarEvolution.h
  llvm/lib/Analysis/ScalarEvolution.cpp
  llvm/test/Analysis/ScalarEvolution/pr62380.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D149251.517884.patch
Type: text/x-patch
Size: 6665 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230428/3e2b9b70/attachment.bin>


More information about the llvm-commits mailing list