You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have the alias universal_host_pinned_memory_resource in each system with a comment that it is an alias to universal_memory_resource, which is a lie. IMO, universal_host_pinned_memory_resource should be an alias to universal_memory_resource except for the CUDA system. The current state is inconsistent, because the pointer type of universal_host_pinned_memory_resource is a native pointer and a not a universal_ptr<T>, like for universal_memory_resource. Fixing this is technically a breaking change.
bdice
changed the title
Thrust universal_host_pinned_memory_resource's pointer type should probably be a universial_ptr
Thrust universal_host_pinned_memory_resource's pointer type should probably be a universal_ptrJan 16, 2025
We have the alias
universal_host_pinned_memory_resource
in each system with a comment that it is an alias touniversal_memory_resource
, which is a lie. IMO,universal_host_pinned_memory_resource
should be an alias touniversal_memory_resource
except for the CUDA system. The current state is inconsistent, because the pointer type ofuniversal_host_pinned_memory_resource
is a native pointer and a not auniversal_ptr<T>
, like foruniversal_memory_resource
. Fixing this is technically a breaking change.We should address this in the next major release.
Originally posted by @bernhardmgruber in #2653 (comment)
The text was updated successfully, but these errors were encountered: