-
Notifications
You must be signed in to change notification settings - Fork 75
[0.9] Unified memory allocations #632
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
Conversation
Your PR no longer requires formatting changes. Thank you for your contribution! |
Errors unrelated? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you implement the cpu.jl changes as well. The CPU backend does support unified by default.
Errors unrelated should be gtg. |
The docs error is introduced. You need to add |
Because of my inclusion of a reference to it in another docstring. Fixed. |
See #630
Feedback from #630 has been addressed. When not implemented in the backend, no kwarg or
unified=false
just work, whileunified=true
will fail unless implemented.