-
Notifications
You must be signed in to change notification settings - Fork 113
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
Declare predefined dpcpp_default
and dpcpp fpga
policies as const
#1671
Declare predefined dpcpp_default
and dpcpp fpga
policies as const
#1671
Conversation
…dpcpp_default as const Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
…dpcpp_fpga as const Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
dpcpp_default
and dpcpp fpga
policies as const
…cpp_fpga now declared as const Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
…t and dpcpp_fpga now declared as const" This reverts commit c1c778f.
We will need to update the docs if this change happens. I did a quick search and found that we cite device_policy in the following pages: Execution Policies, Macros, Pass Data to Algorithms. We cite fpga_policy in the following pages: Execution Policies and Macros. As long as I have guidance, I will update the docs as needed. |
…p_fpga_as_const Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com> # Conflicts: # include/oneapi/dpl/pstl/hetero/dpcpp/execution_sycl_defs.h
@dcbenito, @SergeyKopienko - I have added the documentation changes to the pull request, please take a look. |
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.
I have one change, but otherwise LGTM.
@@ -94,8 +94,8 @@ Macro Description | |||
such as ``dpcpp_default`` and ``dpcpp_fpga``. When the macro is not defined (by default) | |||
or evaluates to non-zero, predefined policies objects can be used. | |||
When the macro is set to 0, predefined policies objects and make functions | |||
without arguments, when ``make_device_policy()``, | |||
``make_fpga_policy()``, are not available. | |||
without arguments - ``make_device_policy()`` and |
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.
Please change the dashes - to parentheses (): "without arguments (make_device_policy()
and make_fpga_policy()
) are not available."
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.
Fixed
…hange the dashes - to parentheses (): "without arguments (make_device_policy() and make_fpga_policy()) are not available." Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
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
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, as discussed, lets wait for the spec change to go through before merging.
I have committed the changes to the specification: uxlfoundation/oneAPI-spec#554 |
In this PR Declare predefined
dpcpp_default
anddpcpp fpga
policies asconst
.I hope it's may help to implement move-constructor and move-assignment in the PR #1652 properly.
PR with the changes for oneDPL specification: uxlfoundation/oneAPI-spec#554