-
Notifications
You must be signed in to change notification settings - Fork 407
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
Let SYCL USMObjectMem use SharedAllocationRecord #3898
Conversation
760f270
to
7e596e7
Compare
Retest this please. |
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.
While I'm never thrilled with moving template definitions to .cpp files and explicit instantiations, it makes sense here.
7e596e7
to
8021926
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.
@dalg24 I addressed your comments (and rebased).
Based on top of #3873.
To avoid circular dependencies I needed to move the definitions using
SharedAllocationRecord
to the*.cpp
file. After moving, this pull request only replacessycl::malloc
withRecord::allocate
andsycl::free
withRecord::decfement
.