-
Notifications
You must be signed in to change notification settings - Fork 22
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
[ready] kokkos impl gemm
#197
Conversation
tpl-implementations/include/experimental/__p1673_bits/kokkos-kernels/blas3_gemm_kk.hpp
Outdated
Show resolved
Hide resolved
tpl-implementations/include/experimental/__p1673_bits/kokkos-kernels/blas3_gemm_kk.hpp
Outdated
Show resolved
Hide resolved
@mhoemmen i have updated this quite a bit, let me know what you think, when you a have a chance! I know you are busy :) |
...plementations/include/experimental/__p1673_bits/kokkos-kernels/blas3_overwriting_gemm_kk.hpp
Outdated
Show resolved
Hide resolved
...plementations/include/experimental/__p1673_bits/kokkos-kernels/blas3_overwriting_gemm_kk.hpp
Show resolved
Hide resolved
...plementations/include/experimental/__p1673_bits/kokkos-kernels/blas3_overwriting_gemm_kk.hpp
Outdated
Show resolved
Hide resolved
...plementations/include/experimental/__p1673_bits/kokkos-kernels/blas3_overwriting_gemm_kk.hpp
Show resolved
Hide resolved
tpl-implementations/include/experimental/__p1673_bits/kokkos-kernels/blas3_updating_gemm_kk.hpp
Outdated
Show resolved
Hide resolved
...mplementations/include/experimental/__p1673_bits/kokkos-kernels/mdspan_to_view_mapper_kk.hpp
Outdated
Show resolved
Hide resolved
auto kokkos_p = to_kokkos_pointer(a.data()); | ||
using view_type = Kokkos::View< | ||
decltype(kokkos_p)*, | ||
typename LayoutMapper<typename mdspan_type::layout_type>::type | ||
decltype(kokkos_p)*, typename LayoutMapper<NestedLayout>::type |
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.
decltype(kokkos_p)*, typename LayoutMapper<NestedLayout>::type | |
std::add_pointer_t<decltype(kokkos_p)>, typename LayoutMapper<NestedLayout>::type |
(This converts, e.g., int*
to int**
, and const int*
to const int**
.)
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.
why is this wrong? this is for a rank-2 mdspan, so it becomes a rank-2 view
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.
@fnrizzi It's not wrong, just a suggestion : - )
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.
ok! suggestion taken!
@fnrizzi To clarify: It's fine to merge now, but you might like to fix the typos first. If you're in a hurry, just let me know and I'll merge it. |
@mhoemmen no hurry, i will fix this properly! |
auto kokkos_p = to_kokkos_pointer(a.data()); | ||
using view_type = Kokkos::View< | ||
decltype(kokkos_p)*, | ||
typename LayoutMapper<typename mdspan_type::layout_type>::type | ||
decltype(kokkos_p)*, typename LayoutMapper<NestedLayout>::type |
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.
@fnrizzi It's not wrong, just a suggestion : - )
No description provided.