[PATCH] D61149: [DAGCombiner] try repeated fdiv divisor transform before building estimate

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 1 19:46:14 PDT 2019


nemanjai accepted this revision.
nemanjai added a comment.

Interestingly enough, I tried this (or some variant of this) and it didn't work for me. Clearly I had done something wrong when trying it. In any case, I've run this against the code we were originally spinning on and it's all good. LGTM and thanks for fixing so quickly.


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

https://reviews.llvm.org/D61149





More information about the llvm-commits mailing list