[PATCH] D27216: [SCEVExpand] do not hoist divisions by zero (PR30935)

[Sebastian Pop via Phabricator via llvm-commits]

sebpop added a comment.

In https://reviews.llvm.org/D27216#619902, @atrick wrote:

> How do you know this won't move division that used to be outside the loop (possibly multiple levels) inside the innermost loop?


You are right: expanding SCEV expressions could lead to a loss in performance.
I'm not sure we have all the information we need to generate code at least as efficient as the one we started from.


Repository:
  rL LLVM

https://reviews.llvm.org/D27216