-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add functions to wrap headers #73
Conversation
Do you have a branch of rocPRIM or rocBLAS with this integrated? |
Working on getting both with the latest changes, yes. |
rocPRIM: https://github.com/lawruble13/rocPRIM/tree/reorg-rocm-cmake Still needs documentation and a changelog update, but I wanted to start gathering feedback. |
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.
Based on validation results/updates from new api usages in rocblas, rocprim, All observations are taken care in latest version.
LGTM to get this merged.
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.
If you've tested the most recent version of your code against one of the existing libraries and you're satisfied with it, then I think it's ready to merge. Nice work, Liam.
No description provided.