[PATCH] D129745: Fix a stack overflow in ScalarEvolution.

[Johannes Reifferscheid via Phabricator via llvm-commits]

jreiffers added a comment.

In D129745#3651133 <https://reviews.llvm.org/D129745#3651133>, @nikic wrote:

> Before doing something like this, we should evaluate compile-time impact of dropping this optimization entirely.

Happy to do that, but some tests appear to depend on this. programUndefinedIfPoison looks fairly expensive on its own. I'm fairly new to this project, do you have a pointer what to run?

It would also be great if I could drop the optimization in a follow-up, since this bug is currently causing production issues for us (and I'm not sure how long the tests will take to fix).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129745