[PATCH] D29897: [BypassSlowDivision] Use ValueTracking to simplify run-time checks

Nikolai Bozhenov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 05:52:15 PST 2017


n.bozhenov added a comment.

In https://reviews.llvm.org/D29897#681767, @jlebar wrote:

> Do you think it would be too conservative for us to say: We won't optimize a long div if you do a short div with the same operands anywhere in the same function?


That's an interesting heuristic, but I believe it's beyond the scope of this particular patch. This patch just takes advantage of ValueTracking and simplifies runtime checks where possible.


https://reviews.llvm.org/D29897





More information about the llvm-commits mailing list