[PATCH] D96647: [BasicAA] Add depth limit

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 18:43:44 PST 2021


reames added a comment.

In D96647#2561923 <https://reviews.llvm.org/D96647#2561923>, @nikic wrote:

> Though now that I think on it, it might make more sense to restrict the total number of recursive queries rather than the query depth in particular, which could allow either a deep linear query, or a shallow branching one.

+1 to this thought.  Are you working on a patch in this direction?  I'm interested in seeing this resolved so that D92401 <https://reviews.llvm.org/D92401> can be reapplied.  I've got a test case that needs something along the lines of D92401 <https://reviews.llvm.org/D92401> to get decent results, and I don't see any good workarounds.  If you need help or motivated review, let me know.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96647



More information about the llvm-commits mailing list