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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 1 09:44:16 PDT 2019


spatel reopened this revision.
spatel added a comment.
This revision is now accepted and ready to land.

Reopening - I reverted this at rL359695 <https://reviews.llvm.org/rL359695> because it can cause an infinite loop from opposing combines.
There's a proposal to solve that in D61384 <https://reviews.llvm.org/D61384>, so I'll merge that here and try again.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61149





More information about the llvm-commits mailing list