-
Notifications
You must be signed in to change notification settings - Fork 51
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
Mention oneDPL TPL #496
Mention oneDPL TPL #496
Conversation
@@ -134,6 +134,8 @@ These defines give information about what third-party libaries Kokkos was compil | |||
+-------------------------------+-----------------------------------------------------------------------------------------------------------------------+ | |||
| ``KOKKOS_ENABLE_LIBQUADMATH`` | Defined if Kokkos links to the `GCC Quad-Precision Math Library API <https://gcc.gnu.org/onlinedocs/libquadmath/>`_. | | |||
+-------------------------------+-----------------------------------------------------------------------------------------------------------------------+ | |||
| ``KOKKOS_ENABLE_ONEDPL`` | Defined if Kokkos links to the `oneDPL library <https://github.com/oneapi-src/oneDPL>`_ when using the SYCL backend. | | |||
+-------------------------------+-----------------------------------------------------------------------------------------------------------------------+ |
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.
Should we expand to say what it is used for and that things like Kokkos::sort
might be less performant if not found?
It is fine if you want to do as a follow up.
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.
3ca94ed
to
d02c302
Compare
d02c302
to
c2725fe
Compare
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.
Looks good to me!
@@ -225,6 +225,9 @@ The following options control enabling TPLs: | |||
* * ``Kokkos_ENABLE_LIBDL`` | |||
* Whether to enable the LIBDL library | |||
* ``ON`` | |||
* * ``Kokkos_ENABLE_ONEDPL`` | |||
* Whether to enable the oneDPL library | |||
* ``ON`` |
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.
No language about it being a SYCL-specific TPL?
Fixes #494.