libcudacxx icon indicating copy to clipboard operation
libcudacxx copied to clipboard

Polymorphic Allocator Proposal

Open jrhemstad opened this issue 3 years ago • 3 comments

Design: #129

Depends on #105

Building of off cuda::memory_resource to introduce cuda::polymorphic_allocator-like types

jrhemstad avatar Feb 08 '21 20:02 jrhemstad

I'd really rather not call it polymorphic. It's a bad name, it's not important for the user to know that, and some classes of performance-sensitive users are allergic to polymorphism even if it's not relevant to performance (as is the case here).

brycelelbach avatar Feb 27 '21 02:02 brycelelbach

I'd really rather not call it polymorphic. It's a bad name, it's not important for the user to know that, and some classes of performance-sensitive users are allergic to polymorphism even if it's not relevant to performance (as is the case here).

I have no qualms with axing the "polymorphic" part of the name. Would we just call it cuda::allocator and it always works with a memory_resource*?

jrhemstad avatar Mar 01 '21 16:03 jrhemstad

I'm in favor of cuda::allocator.

brycelelbach avatar Mar 04 '21 07:03 brycelelbach

Superseded by https://github.com/NVIDIA/libcudacxx/pull/309

jrhemstad avatar Feb 20 '23 16:02 jrhemstad