[PATCH] D65898: [InstCombine] x /c fabs(x) -> copysign(1.0, x)

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 marked 4 inline comments as done.
xbolva00 added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineMulDivRem.cpp:1240
+      match(Op1, m_Intrinsic<Intrinsic::fabs>(m_Specific(Op0))) &&
+      Op1->hasOneUse()) {
+    Value *V = Builder.CreateBinaryIntrinsic(
----------------
spatel wrote:
> Do we need the one-use restriction?
Dropped.


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

https://reviews.llvm.org/D65898