-
Notifications
You must be signed in to change notification settings - Fork 64
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
#270: fix missing qualifiers in compressed_pair #271
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.
Lets catch the C++14 not supported in make
Pushed these changes |
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 do we do the C++14 in the same PR?
That does not seem like a good idea
What prevents you to just qualify _MDSPAN_TRAIT(std::is_empty, _T)
or whatnot
171be58
to
4ba7fbe
Compare
I made the PR C++14 compatible |
Make sure you update the title line and description |
* #6161: update mdspan and fix namespaces. Also import from std namespace if using system mdspan * #6161: fix formatting * #6161: apply mdspan changes from kokkos/mdspan#271 * #6161: make namespace qualification consistent * #6161: fix last inconsistent namespace qual * #6161: fix formatting * #6161: use old C++14-compatible traits * #6161: copy latest mdspan stable with maybe_static_array bug fix * #6161: don't snapshot the stdmode headers as we don't use them and it can cause only conflicts * #6161: add -DKokkos_ENABLE_IMPL_MDSPAN=ON config to the generic github ci build, nvhpc, and two cuda ci builds * #6161: use feature test macro instead of presence of header to avoid pulling in partial mdspan implementations * #6161: use mdspan, layout_*, and default_accessor from std namespace if using compiler-provided mdspan * #6161: fix formatting * #6161: guard <version> include with c++20 or above * #6161: disable standard mdspan if using CUDA or HIP since it may not be compatible * #6161: format fix * Work around CUDA 11.0 deduction failure for tuple * Do not use mdspan from compiler by default even if header is detected * #6161: make snapshot consistent with mdspan --------- Co-authored-by: Christian Trott <crtrott@sandia.gov>
* kokkos#6161: update mdspan and fix namespaces. Also import from std namespace if using system mdspan * kokkos#6161: fix formatting * kokkos#6161: apply mdspan changes from kokkos/mdspan#271 * kokkos#6161: make namespace qualification consistent * kokkos#6161: fix last inconsistent namespace qual * kokkos#6161: fix formatting * kokkos#6161: use old C++14-compatible traits * kokkos#6161: copy latest mdspan stable with maybe_static_array bug fix * kokkos#6161: don't snapshot the stdmode headers as we don't use them and it can cause only conflicts * kokkos#6161: add -DKokkos_ENABLE_IMPL_MDSPAN=ON config to the generic github ci build, nvhpc, and two cuda ci builds * kokkos#6161: use feature test macro instead of presence of header to avoid pulling in partial mdspan implementations * kokkos#6161: use mdspan, layout_*, and default_accessor from std namespace if using compiler-provided mdspan * kokkos#6161: fix formatting * kokkos#6161: guard <version> include with c++20 or above * kokkos#6161: disable standard mdspan if using CUDA or HIP since it may not be compatible * kokkos#6161: format fix * Work around CUDA 11.0 deduction failure for tuple * Do not use mdspan from compiler by default even if header is detected * kokkos#6161: make snapshot consistent with mdspan --------- Co-authored-by: Christian Trott <crtrott@sandia.gov>
* kokkos#6161: update mdspan and fix namespaces. Also import from std namespace if using system mdspan * kokkos#6161: fix formatting * kokkos#6161: apply mdspan changes from kokkos/mdspan#271 * kokkos#6161: make namespace qualification consistent * kokkos#6161: fix last inconsistent namespace qual * kokkos#6161: fix formatting * kokkos#6161: use old C++14-compatible traits * kokkos#6161: copy latest mdspan stable with maybe_static_array bug fix * kokkos#6161: don't snapshot the stdmode headers as we don't use them and it can cause only conflicts * kokkos#6161: add -DKokkos_ENABLE_IMPL_MDSPAN=ON config to the generic github ci build, nvhpc, and two cuda ci builds * kokkos#6161: use feature test macro instead of presence of header to avoid pulling in partial mdspan implementations * kokkos#6161: use mdspan, layout_*, and default_accessor from std namespace if using compiler-provided mdspan * kokkos#6161: fix formatting * kokkos#6161: guard <version> include with c++20 or above * kokkos#6161: disable standard mdspan if using CUDA or HIP since it may not be compatible * kokkos#6161: format fix * Work around CUDA 11.0 deduction failure for tuple * Do not use mdspan from compiler by default even if header is detected * kokkos#6161: make snapshot consistent with mdspan --------- Co-authored-by: Christian Trott <crtrott@sandia.gov>
* kokkos#6161: update mdspan and fix namespaces. Also import from std namespace if using system mdspan * kokkos#6161: fix formatting * kokkos#6161: apply mdspan changes from kokkos/mdspan#271 * kokkos#6161: make namespace qualification consistent * kokkos#6161: fix last inconsistent namespace qual * kokkos#6161: fix formatting * kokkos#6161: use old C++14-compatible traits * kokkos#6161: copy latest mdspan stable with maybe_static_array bug fix * kokkos#6161: don't snapshot the stdmode headers as we don't use them and it can cause only conflicts * kokkos#6161: add -DKokkos_ENABLE_IMPL_MDSPAN=ON config to the generic github ci build, nvhpc, and two cuda ci builds * kokkos#6161: use feature test macro instead of presence of header to avoid pulling in partial mdspan implementations * kokkos#6161: use mdspan, layout_*, and default_accessor from std namespace if using compiler-provided mdspan * kokkos#6161: fix formatting * kokkos#6161: guard <version> include with c++20 or above * kokkos#6161: disable standard mdspan if using CUDA or HIP since it may not be compatible * kokkos#6161: format fix * Work around CUDA 11.0 deduction failure for tuple * Do not use mdspan from compiler by default even if header is detected * kokkos#6161: make snapshot consistent with mdspan --------- Co-authored-by: Christian Trott <crtrott@sandia.gov>
Closes #270