-
Notifications
You must be signed in to change notification settings - Fork 72
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
Deprecating hipblasComplex types and adding _v2 version of ixamax #656
Conversation
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 seems to match the other v2 pattern. Minor suggestion made
} | ||
else | ||
{ | ||
EXPECT_EQ(HIPBLAS_STATUS_SUCCESS, status); // fail |
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.
All the // fail comments not really sure of.
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.
Yeah, that line just ensures that we actually get a gtest failure when we don't get HIPBLAS_STATUS_SUCCESS from testing_*.hpp. // fail
is meant to imply that if we get to that line, the test will fail. hipBLAS gtest framework needs some rework.
Beginning of deprecation of hipblas complex types. Just made changes for ixamax for now, please let me know if you had something else in mind for this process.