Skip to content

Conversation

@jinge90
Copy link
Contributor

@jinge90 jinge90 commented Dec 12, 2025

We received request for tanh in bfloat16 math functions. This PR adds it for SPIRV targets using fp32 emulation and update extension doc.

@jinge90 jinge90 requested review from a team as code owners December 12, 2025 07:57
@jinge90 jinge90 requested a review from xtian-github December 12, 2025 08:01
@jinge90
Copy link
Contributor Author

jinge90 commented Dec 12, 2025

Hi, @xtian-github
This trivial PR is for adding tanh in bf16 math for SPIRV targets.

Signed-off-by: jinge90 <ge.jin@intel.com>
@jinge90
Copy link
Contributor Author

jinge90 commented Dec 13, 2025

Hi, @intel/dpcpp-specification-reviewers
Could you take a look at this PR?
Thanks very much!

@jinge90
Copy link
Contributor Author

jinge90 commented Dec 15, 2025

Hi, @intel/dpcpp-specification-reviewers
Kind ping~
Thanks very much.

Copy link
Contributor

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, though I wonder if there's a reason we don't offer tan, sinh and cosh?

@github-actions
Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

@jinge90
Copy link
Contributor Author

jinge90 commented Dec 15, 2025

LGTM, though I wonder if there's a reason we don't offer tan, sinh and cosh?

I have no idea about this but think we can add these functions as well.
Thanks very much!

@jinge90
Copy link
Contributor Author

jinge90 commented Dec 16, 2025

Hi, @intel/llvm-gatekeepers
Could you help merge this PR, we need it ASAP.
Thanks very much.

@bader bader merged commit 34748e0 into intel:sycl Dec 16, 2025
50 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants