[PATCH] D50913: [FPEnv] Don't need copysign/fabs/fneg constrained intrinsics

[Cameron McInally via Phabricator via llvm-commits]

cameron.mcinally added a comment.

It looks like I spoke too soon. We DO need a constrained fneg intrinsic. LLVM does not have a dedicated FNEG operator, but rather uses SUB. This is not okay since 0-x can trap, where -x cannot trap. I'll create a separate patch and Differential for FNEG.

Besides that, we still don't need constrained copysign/fabs intrinsics...


Repository:
  rL LLVM

https://reviews.llvm.org/D50913