Add minor fixes to accessor/multi_ptr wording #437
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some minor issues were raised in #432 after the PR was merged, so this is a follow-up PR to address those.
This PR adds a restriction for
accessor::get_multi_ptr
that it's only available whenAccessTarget
istarget::device
as it can't be used in a host task.It adds the
value_type
alias to thedecorated::legacy
specialization ofmulti_ptr
and adds theget_raw
andget_decorated
member functions to thevoid
/decorated::legacy
specializations ofmulti_ptr
.It also changes the alias used in
multi_ptr::get_raw
in thedecorated::legacy
specialization toElementType
asvalue_type
is not defined.cc @gmlueck @steffenlarsen