-
Notifications
You must be signed in to change notification settings - Fork 407
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
Provide ViewCtorProp overload for create_mirror_view_and_copy #5125
Provide ViewCtorProp overload for create_mirror_view_and_copy #5125
Conversation
887b62a
to
f363abb
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.
Just not sure about that cast
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.
Ok its fine, cast is making use of inheritance.
46976fb
to
375a21f
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 reasonable, although the change to TEST_EXECSPACE
in 6ce5a9 could be done in a separate pr (including other tests in containers that might benefit)
Similar to #5095, #5035 and #4844. #4826 would help remove more fences here.
This pull request provides a
ViewCtorProp
overload forcreate_mirror_view_and_copy
that also allows specifying an execution space to be used in theallocation
and thedeep_copy
. The existing version is changed to always use an execution space forallocation
(but still uses the two-argumentdeep_copy
).