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.

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 spmv transpose (Fix #829) #831

Merged
merged 2 commits into from
Oct 14, 2020

Conversation

brian-kelley
Copy link
Contributor

@brian-kelley brian-kelley commented Oct 14, 2020

Add tests for SpMV modes conjugate (C), transpose (T) and conjugate transpose (H). The new tests use the same matrix as the existing normal (N) mode test. Added 1D and 2D vector versions but didn't touch the structured spmv tests, they may still need more tests.

#######################################################
PASSED TESTS
#######################################################
clang-8.0-Cuda_OpenMP-release build_time=812 run_time=108
clang-8.0-Pthread_Serial-release build_time=223 run_time=154
clang-9.0.0-Pthread-release build_time=133 run_time=67
clang-9.0.0-Serial-release build_time=141 run_time=52
cuda-10.1-Cuda_OpenMP-release build_time=918 run_time=130
cuda-11.0-Cuda_OpenMP-release build_time=1157 run_time=1177
cuda-9.2-Cuda_Serial-release build_time=943 run_time=196
gcc-7.3.0-OpenMP-release build_time=156 run_time=257
gcc-7.3.0-Pthread-release build_time=129 run_time=63
gcc-8.3.0-Serial-release build_time=153 run_time=49
gcc-9.1-OpenMP-release build_time=193 run_time=101
gcc-9.1-Serial-release build_time=171 run_time=51
intel-17.0.1-Serial-release build_time=307 run_time=59
intel-18.0.5-OpenMP-release build_time=392 run_time=50
intel-19.0.5-Pthread-release build_time=446 run_time=71

Copy link
Contributor

@lucbv lucbv left a comment

Choose a reason for hiding this comment

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

Yeah, this looks fine to me

@brian-kelley
Copy link
Contributor Author

@lucbv Thanks for reviewing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants