[PATCH] D79787: [IndVarSimplify][LoopUtils] Avoid TOCTOU/ordering issues (PR45835)

Fri May 15 10:51:36 PDT 2020

lebedev.ri added a comment.

In D79787#2031793 <https://reviews.llvm.org/D79787#2031793>, @dmajor wrote:

> Thanks a lot for taking this over! Looks good from my side but you may wish to wait for a more expert review.
>
> I also confirmed that this still fixes our issue on the 10.0 branch. I had to remove `-scev-cheap-expansion-budget=1` since that didn't exist in 10, but fortunately it's not needed there. Since it wasn't a clean merge, I put the diff in P8220 <https://reviews.llvm.org/P8220> if it makes your/Tom's life easier.

Thank you for the review!
Waiting on @reames / @mkazantsev / ...

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79787