[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