[PATCH] D62699: [InlineCost] Add support for UnaryOperator

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/Analysis/InlineCost.cpp:1131
+  // is expensive, this may eventually become a library call. Treat the cost
+  // as such.
+  if (I.getType()->isFloatingPointTy() &&
----------------
Is this actually true?  fneg should be one or two native instructions for almost every combination of type/target, even ones that don't have native floating-point support.


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

https://reviews.llvm.org/D62699