[PATCH] D139309: InstCombine: Fold fabs (copysign x, y) -> fabs x

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/IR/IRBuilder.h:946
+                           Instruction *FMFSource = nullptr,
+                           const Twine &Name = "") {
+    return CreateBinaryIntrinsic(Intrinsic::copysign, LHS, RHS, FMFSource,
----------------
kpn wrote:
> This isn't used? Is it needed yet?
I have a stack of copysign patches it's used somewhere. I'll split this out to first use before pushing


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

https://reviews.llvm.org/D139309