[PATCH] D136098: InstCombine: Fold fdiv nnan x, 0 -> copysign(inf, x)

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp:1375
 /// Remove negation and try to convert division into multiplication.
-static Instruction *foldFDivConstantDivisor(BinaryOperator &I) {
+Instruction *InstCombinerImpl::foldFDivConstantDivisor(BinaryOperator &I) {
   Constant *C;
----------------
kpn wrote:
> Is this part of a future change? I'm wondering if it belongs in this patch.
No, this needed to move into the class for replaceInstUsesWith


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

https://reviews.llvm.org/D136098