[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