[PATCH] D67207: [JumpThreading] Fix the AssertVH handling for debug builds.

Thu Sep 5 19:05:14 PDT 2019

skatkov added a comment.

Thank you guys, for feedback.

I found this issue fighting against the crash in jump-threading. It is in progress now and goes slowly due to intermittent behavior.
The current guess is related to broken DT and LoopInfo by jump threading pass but it uses alias analysis which uses Loop->getExitEdges(). So I tried to move from lazy update of DT  to eager and found this issue, so posted the patch.
After that I found that it does not resolve the issue with eager DT as I wrote before.

I agree that this patch does not resolve the issue.
I will try to return to  return to this one later when I have cycles. Functional crash has more priority for me now.
And I do not have clear reproducer at this moment.

So, thank you again and sorry if I bothered you.

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

https://reviews.llvm.org/D67207