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

Sebastian Pop via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 12 10:58:45 PST 2016

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.



More information about the llvm-commits mailing list