[PATCH] D146905: [IR] Add llvm.tan.* intrinsic

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 10:50:01 PDT 2023


kpn added a comment.

I don't see any tests for a constrained/STRICT tan. We do have tests for constrained sin and cos so tan tests where sin and cos are tested is probably appropriate.



================
Comment at: llvm/docs/LangRef.rst:14560
+using a less accurate calculation.
+
 '``llvm.pow.*``' Intrinsic
----------------
If you are adding a constrained/STRICT tan then you need to document it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146905



More information about the llvm-commits mailing list