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

Missing definition for allocate_managed() in src/cuda/api/context.hpp #455

Closed
kingcrimsontianyu opened this issue Jan 18, 2023 · 1 comment

Comments

@kingcrimsontianyu
Copy link

As the title states. The same issue exists for allocate().

@eyalroz eyalroz added the bug label Jan 18, 2023
eyalroz added a commit that referenced this issue Jan 18, 2023
@eyalroz
Copy link
Owner

eyalroz commented Jan 18, 2023

@kingcrimsontianyu : Fixed - try the development branch. Sorry, that one slipped by me somehow. TBH, the repository is really missing unit test coverage (and the example programs don't cover that much, see #18). But I just don't have the time to sit down and write all of that - with this being a spare-time project only.

As for allocate() - it's actually implemented.

For v0.6.1, you can use the stand-alone function as a workaround:

cuda::memory::managed::allocate(my_context, 1234, my_visibility_setting);

eyalroz added a commit that referenced this issue Feb 10, 2023
eyalroz added a commit that referenced this issue Mar 9, 2023
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

2 participants