-
Notifications
You must be signed in to change notification settings - Fork 129
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
test for svm allocator #226
test for svm allocator #226
Conversation
79f4cd2
to
eb1d981
Compare
74d0a25
to
e847377
Compare
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.
Thanks! Just one last comment. Also needs a rebase (hopefully trivial).
e847377
to
a3d2ee2
Compare
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.
LGTM, just waiting for CI to complete (which has been a problem recently...).
a3d2ee2
to
bd62aae
Compare
@bashbaug CI failed because of random stuff- some CI internal errors probably. |
bd62aae
to
323249d
Compare
Merging as discussed in the May 30th teleconference. |
I wrote some code to test allocate and deallocate API (clSVMAlloc, clSVMFree, clEnqueueSVMMap)