-
Notifications
You must be signed in to change notification settings - Fork 733
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SYCL][DOC] DeviceLib doc update to add new supported items #3623
Conversation
Signed-off-by: gejin <ge.jin@intel.com>
Signed-off-by: gejin <ge.jin@intel.com>
@jinge90, could you remind me PR/commit IDs where these functions were added, please? |
Hi, @bader And a bunch of math functions are removed from list of math function unsupported although we didn't add any support in SYCL device library. The reason is those math functions are converted to corresponding llvm intrinsic and llvm-spirv has supported those intrinsic, so they can work in SYCL device now, The corresponding PR is: KhronosGroup/SPIRV-LLVM-Translator#855 |
Thanks, but I can't find |
Hi, @bader For nextafter, I think it was a mistake in initial version of the doc, this API was added long time ago in the first patch for cmath, complex device library, #975 |
Hi, @intel/dpcpp-specification-reviewers |
Add @pvchupin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Signed-off-by: gejin ge.jin@intel.com