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.

Sign up for GitHub

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

Jump to bottom

[SPARSE] Fix oneMKL name usages in comments #587

Merged
merged 2 commits into from
Oct 14, 2024
Merged

[SPARSE] Fix oneMKL name usages in comments #587

merged 2 commits into from
Oct 14, 2024

Conversation

dnhsieh-intel
Copy link
Contributor

Description

This PR is a small fix of oneMKL name usages.

Checklist

All Submissions

  • Do all unit tests pass locally? Only comments are updated.
  • Have you formatted the code using clang-format?

@dnhsieh-intel dnhsieh-intel requested a review from a team October 10, 2024 03:52
@dnhsieh-intel dnhsieh-intel changed the title [SPARSE] Fix oneMKL usages in comments [SPARSE] Fix oneMKL name usages in comments Oct 10, 2024
Rbiessy
Rbiessy reviewed Oct 10, 2024
View reviewed changes
Copy link
Contributor

@Rbiessy Rbiessy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry about this, thanks for the PR

src/sparse_blas/backends/mkl_common/mkl_handles.hpp Outdated Show resolved Hide resolved
@dnhsieh-intel dnhsieh-intel merged commit d616d51 into oneapi-src:develop Oct 14, 2024
6 checks passed
@dnhsieh-intel dnhsieh-intel deleted the dev/dainihsi/onemkl branch October 15, 2024 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rbiessy Rbiessy Rbiessy approved these changes

@mkrainiuk mkrainiuk mkrainiuk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dnhsieh-intel @mkrainiuk @Rbiessy