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

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 28 09:46:39 PST 2018


cameron.mcinally abandoned this revision.
cameron.mcinally added a comment.

This issue was handled with the addition of a dedicated FNeg IR Instruction in D53877 <https://reviews.llvm.org/D53877>.

At this time, I do not know of a need for constrained fabs/fneg/fcopysign intrinsics. We can reevaluate if unsafe transformations are found in the future.

Thanks for the contributions, everyone.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D50913





More information about the llvm-commits mailing list