Skip to content
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

Make memory::region_t CUDA-independent #604

Closed
eyalroz opened this issue Mar 10, 2024 · 0 comments
Closed

Make memory::region_t CUDA-independent #604

eyalroz opened this issue Mar 10, 2024 · 0 comments

Comments

@eyalroz
Copy link
Owner

eyalroz commented Mar 10, 2024

There is only one aspect maiking cuda::memory::region_t depend on CUDA in any way, and that is the constructibility using CUdevice ptr. It is rather painless to take that away, and make the class a pure C++ utility; instead of this ctor, we can use cuda::memory::as_pointer() on the CUdeviceptr, and then construct in oblivion of CUDA.

eyalroz added a commit that referenced this issue Mar 10, 2024
* `memory::region_t`'s are now CUDA-independent!
* `memory::const_region_t's` constructor from `region_t`'s is now by value rather than by const reference, so that rvalue-references are also supported
eyalroz added a commit that referenced this issue Mar 10, 2024
* `memory::region_t`'s are now CUDA-independent!
* `memory::const_region_t's` constructor from `region_t`'s is now by value rather than by const reference, so that rvalue-references are also supported
eyalroz added a commit that referenced this issue Mar 10, 2024
* `memory::region_t`'s are now CUDA-independent!
* `memory::const_region_t's` constructor from `region_t`'s is now by value rather than by const reference, so that rvalue-references are also supported
eyalroz added a commit that referenced this issue Mar 10, 2024
* `memory::region_t`'s are now CUDA-independent!
* `memory::const_region_t's` constructor from `region_t`'s is now by value rather than by const reference, so that rvalue-references are also supported
eyalroz added a commit that referenced this issue Apr 20, 2024
* `memory::region_t`'s are now CUDA-independent!
* `memory::const_region_t's` constructor from `region_t`'s is now by value rather than by const reference, so that rvalue-references are also supported
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant