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

[Cameron McInally via Phabricator via llvm-commits]

cameron.mcinally added a comment.

So LangRef says that there is no explicit fneg:

"Note that the ‘fsub’ instruction is used to represent the ‘fneg’ instruction present in most other intermediate representations."

But this doesn't seem to be the case in IRBuilder. I.e.: CreateFNeg(...)

Perhaps separating the FNeg and FSub operators is a good solution. And also removing all the xforms between the two. That makes more sense since fneg really *shouldn't* be constrained (it's just a bitwise operation).


Repository:
  rL LLVM

https://reviews.llvm.org/D50913