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

[Justin Lebar via Phabricator via llvm-commits]

jlebar added a comment.

In https://reviews.llvm.org/D29897#682146, @n.bozhenov wrote:

> 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.


Sure.  I would like not to lose track of this idea, though.


https://reviews.llvm.org/D29897