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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 08:35:39 PDT 2022


fhahn added a comment.

In D129745#3651135 <https://reviews.llvm.org/D129745#3651135>, @jreiffers wrote:

> 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. programUndefinedIfPoison looks fairly expensive on its own. I'm fairly new to this project, do you have a pointer what to run? Also, unfortunately some tests appear to depend on this optimization.
>
> 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).

Would it be possible to share a reproducer? Is this a new stack overflow?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129745



More information about the llvm-commits mailing list