diff --git a/containers/unit_tests/TestDynRankViewTypedefs.cpp b/containers/unit_tests/TestDynRankViewTypedefs.cpp index f6aff146e75..81cda721add 100644 --- a/containers/unit_tests/TestDynRankViewTypedefs.cpp +++ b/containers/unit_tests/TestDynRankViewTypedefs.cpp @@ -258,10 +258,10 @@ namespace TestIntAtomic { using expected_ref_type = desul::AtomicRef; #endif // clang-format on - static_assert(test_view_typedefs( - ViewParams>{})); - // clang-format off +static_assert(test_view_typedefs( + ViewParams>{})); +// clang-format off } // clang-format on } // namespace diff --git a/core/src/Kokkos_View.hpp b/core/src/Kokkos_View.hpp index 44ad254f276..5c921fb6ef5 100644 --- a/core/src/Kokkos_View.hpp +++ b/core/src/Kokkos_View.hpp @@ -208,7 +208,7 @@ class View : public Impl::BasicViewFromTraits::type { typename traits::device_type, typename traits::hooks_policy, typename traits::memory_traits>; - // Compatible HostMirror view + // Compatible HostMirror view using host_mirror_type = View::type { size_t> stride(iType r) const { // base class doesn't have constraint - // FIXME: Eventually we need to deprecate this behavior and just use - // BasicView implementation + // FIXME: Eventually we need to deprecate this behavior and just use + // BasicView implementation return base_t::stride(r); } @@ -313,8 +313,7 @@ class View : public Impl::BasicViewFromTraits::type { KOKKOS_INLINE_FUNCTION const Kokkos::Impl::SharedAllocationTracker& impl_track() const { - if constexpr ( traits::is_managed ) - { + if constexpr (traits::is_managed) { return base_t::data_handle().tracker(); } else { return {}; diff --git a/core/src/View/Kokkos_BasicView.hpp b/core/src/View/Kokkos_BasicView.hpp index 6b1966bd552..fec78972c7a 100644 --- a/core/src/View/Kokkos_BasicView.hpp +++ b/core/src/View/Kokkos_BasicView.hpp @@ -105,13 +105,14 @@ class BasicView { public: using mdspan_type = mdspan; - using extents_type = typename mdspan_type::extents_type; - using layout_type = typename mdspan_type::layout_type; - using accessor_type = typename mdspan_type::accessor_type; - using mapping_type = typename mdspan_type::mapping_type; - using element_type = typename mdspan_type::element_type; - using value_type = typename mdspan_type::value_type; - // FIXME: backwards compatibility, should be changed to the same as mdspan index_type + using extents_type = typename mdspan_type::extents_type; + using layout_type = typename mdspan_type::layout_type; + using accessor_type = typename mdspan_type::accessor_type; + using mapping_type = typename mdspan_type::mapping_type; + using element_type = typename mdspan_type::element_type; + using value_type = typename mdspan_type::value_type; + // FIXME: backwards compatibility, should be changed to the same as mdspan + // index_type using index_type = typename mdspan_type::size_type; using size_type = typename mdspan_type::size_type; using rank_type = typename mdspan_type::rank_type; diff --git a/core/unit_test/TestViewTypedefs.cpp b/core/unit_test/TestViewTypedefs.cpp index 15683c122a6..940502489b5 100644 --- a/core/unit_test/TestViewTypedefs.cpp +++ b/core/unit_test/TestViewTypedefs.cpp @@ -272,19 +272,18 @@ namespace TestIntAtomic { std::conditional_t>>; - // clang-format on - static_assert( - test_view_typedefs< - layout_type, space, memory_traits, host_mirror_space, int, +// clang-format on +static_assert(test_view_typedefs< + layout_type, space, memory_traits, host_mirror_space, int, #ifdef KOKKOS_ENABLE_IMPL_VIEW_LEGACY - Kokkos::Impl::AtomicDataElement< - Kokkos::ViewTraits>> + Kokkos::Impl::AtomicDataElement< + Kokkos::ViewTraits>> #else - desul::AtomicRef + desul::AtomicRef #endif - >(ViewParams>{})); - // clang-format off + >(ViewParams>{})); +// clang-format off } // clang-format on } // namespace diff --git a/core/unit_test/cuda/TestCuda_Spaces.cpp b/core/unit_test/cuda/TestCuda_Spaces.cpp index d3a955b168e..39c293d251b 100644 --- a/core/unit_test/cuda/TestCuda_Spaces.cpp +++ b/core/unit_test/cuda/TestCuda_Spaces.cpp @@ -305,8 +305,7 @@ struct TestViewCudaTexture { EXPECT_TRUE( (std::is_same::value)); #else - EXPECT_TRUE( - (std::is_same::value)); + EXPECT_TRUE((std::is_same::value)); #endif EXPECT_TRUE(V::reference_type_is_lvalue_reference); // An ordinary view. @@ -314,7 +313,8 @@ struct TestViewCudaTexture { EXPECT_FALSE(T::reference_type_is_lvalue_reference); // Texture fetch // returns by value. #else - EXPECT_TRUE(T::reference_type_is_lvalue_reference); // Returns by value for now. + EXPECT_TRUE( + T::reference_type_is_lvalue_reference); // Returns by value for now. #endif TestViewCudaTexture self;