-
Notifications
You must be signed in to change notification settings - Fork 107
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
[oneDPL] Create a subsection for is_execution_policy type trait #567
base: main
Are you sure you want to change the base?
Conversation
source/elements/oneDPL/source/parallel_api/execution_policies.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Dan Hoeflinger <dan.hoeflinger@intel.com>
source/elements/oneDPL/source/parallel_api/execution_policies.rst
Outdated
Show resolved
Hide resolved
source/elements/oneDPL/source/parallel_api/execution_policies.rst
Outdated
Show resolved
Hide resolved
source/elements/oneDPL/source/parallel_api/execution_policies.rst
Outdated
Show resolved
Hide resolved
source/elements/oneDPL/source/parallel_api/execution_policies.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Ruslan Arutyunyan <ruslan.arutyunyan@intel.com>
``is_execution_policy<T>`` must have the characteristics of ``std::true_type`` if ``T`` is one of the above specified | ||
or implementation-defined execution policy types, otherwise it must have the characteristics of ``std::false_type``. | ||
Following the C++ Standard, ``is_execution_policy<T>`` does not automatically strip references and cv-qualifiers | ||
from its template argument. [*Note*: Use it with ``std::decay_t<T>`` or similar type transformation utilities. -- *end note*] |
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.
Optional: It is a little strange to have this inline note directly followed by a multi-line note.
I think it may be better to either remove the note descriptor and wrapping and just tag on to the previous sentence: " and is best used in combination with ``std::decay_t'' or similar type transformation utilities"
-or-
We can remove this note from the spec in favor of some similar advice in the oneDPL documentation. We may be able to assume some level of sophistication of someone reading this specification, where the previous sentence implies this note.
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.
Generally LGTM, optional comment about the decay note.
This PR describes oneDPL
is_execution_policy
trait in more detail, and also addresses #558.