[PATCH] D69777: [ConstantFolding] Fold calls to FP remainder function

[Ehud Katz via Phabricator via llvm-commits]

ekatz marked an inline comment as done.
ekatz added inline comments.


================
Comment at: llvm/lib/Analysis/ConstantFolding.cpp:2103
+          APFloat V = Op1->getValueAPF();
+          if (APFloat::opStatus::opOK == V.remainder(Op2->getValueAPF()))
+            return ConstantFP::get(Ty->getContext(), V);
----------------
arsenm wrote:
> Backwards condition
Wanted to be consistent with the coding style on that particular file. See line 2095.
If I change this, should I also change it for "mod()" as well?


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

https://reviews.llvm.org/D69777