[PATCH] D70792: [InstCombine] fold copysign with constant sign argument to (fneg+)fabs

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 27 10:00:03 PST 2019


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

LGTM.  I agree it makes sense to consider fabs+fneg canonical, as opposed to an "expensive" operation like fcopysign.


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

https://reviews.llvm.org/D70792





More information about the llvm-commits mailing list