-
Notifications
You must be signed in to change notification settings - Fork 30
Add missing include to math_utils.hpp
#1899
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
Conversation
Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞 |
Array API standard conformance tests for dpctl=0.19.0dev0=py310hdf72452_239 ran successfully. |
Array API standard conformance tests for dpctl=0.19.0dev0=py310hdf72452_240 ran successfully. |
Seems the issue in internal CI doesn't relate to that PR. |
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.
Thank you @antonwolfy
…8.x/gh-1899 Backport to maintenance 0.18.x/gh 1899
The PR proposes to add missing
sycl/sycl.hpp
include tomath_utils.hpp
header file, becausesycl::half
type and functions fromsycl
namespace are used there.Otherwise the isolated include, like
used by an extension code will produce compilation errors: