Clang format fixes

Author: capi1500

include/dr/mp/alignment.hpp

@@ -11,7 +11,9 @@
 namespace dr::mp {
 
 template <typename T>
-concept has_segments = requires(T &t) { dr::ranges::segments(t); };
+concept has_segments = requires(T &t) {
+  dr::ranges::segments(t);
+};
 
 template <typename T>
 concept no_segments = !has_segments<T>;

test/gtest/mp/CMakeLists.txt

@@ -1,11 +1,11 @@
-# SPDX-FileCopyrightText: Intel Corporation
+#SPDX - FileCopyrightText : Intel Corporation
 #
-# SPDX-License-Identifier: BSD-3-Clause
+#SPDX - License - Identifier : BSD - 3 - Clause
 
 set(CMAKE_INCLUDE_CURRENT_DIR ON)
 
-# tested with a variable number of ranks
-# cmake-format: off
+#tested with a variable number of ranks
+#cmake - format : off
 add_executable(
   mp-tests
   mp-tests.cpp
@@ -17,10 +17,10 @@ add_executable(
   ../common/drop.cpp
   ../common/enumerate.cpp
   ../common/equal.cpp
-#  ../common/exclusive_scan.cpp disabled due to deadlock - DRA-213
+#../ common / exclusive_scan.cpp disabled due to deadlock - DRA - 213
   ../common/fill.cpp
   ../common/for_each.cpp
-#  ../common/inclusive_scan.cpp disabled due to deadlock - DRA-213
+#../ common / inclusive_scan.cpp disabled due to deadlock - DRA - 213
   ../common/iota.cpp
   ../common/iota_view.cpp
   ../common/reduce.cpp
@@ -55,9 +55,11 @@ add_executable(
         wide-halo-2d-3.cpp
 )
 
-# mp-quick-test and mp-quick-test-3-only is for development. By reducing the number of source files, it
-# builds much faster. Change the source files to match what you need to test. It
-# is OK to commit changes to the source file list.
+#mp - quick - test and mp - quick - test - 3 -                                 \
+    only is for development.By reducing the number of source files,            \
+    it
+#builds much faster.Change the source files to match what you need to test.It
+#is OK to commit changes to the source file list.
 add_executable(mp-quick-test
   mp-tests.cpp
   halo.cpp
@@ -66,7 +68,7 @@ add_executable(mp-quick-test-3-only
   mp-tests.cpp
   wide-halo-2d-3.cpp
 )
-# cmake-format: on
+#cmake - format : on
 
 target_compile_definitions(mp-quick-test PRIVATE QUICK_TEST)
 target_compile_definitions(mp-quick-test-3-only PRIVATE QUICK_TEST)
@@ -80,8 +82,8 @@ foreach(test-exec IN ITEMS mp-tests mp-tests-3 mp-quick-test mp-quick-test-3-onl
           "${CMAKE_COMMAND} -E time")
 endforeach()
 
-# tests without --sycl flag will fail on IshmemBackend TODO: make them be
-# running somehow if ENABLE_ISHMEM will be default CI config
+#tests without-- sycl flag will fail on IshmemBackend TODO : make them be
+#running somehow if ENABLE_ISHMEM will be default CI config
 if(NOT ENABLE_ISHMEM)
   add_mp_ctest(NAME mp-quick-test NPROC 1)
   add_mp_ctest(NAME mp-quick-test NPROC 2)
@@ -90,7 +92,7 @@ if(NOT ENABLE_ISHMEM)
   cmake_path(GET MPI_CXX_ADDITIONAL_INCLUDE_DIRS FILENAME MPI_IMPL)
 
   if(NOT MPI_IMPL STREQUAL "openmpi")
-    # MPI_Win_create fails for communicator with size 1
+#MPI_Win_create fails for communicator with size 1
     add_mp_ctest(NAME mp-tests NPROC 1 TIMEOUT 150)
   endif()
   foreach(nproc RANGE 2 4)
@@ -100,15 +102,16 @@ if(NOT ENABLE_ISHMEM)
 endif()
 
 if(ENABLE_SYCL)
-  # DRA-83: Slide isn't complete
+#DRA - 83 : Slide isn't complete
   set(sycl-exclusions *Slide*:ComplexSlide*:)
 
   if(ENABLE_ISHMEM)
-    # Some Halo3 cases don't work on IshmemBackend (work on MPI, see: DRA-84),
-    # Counted.large fails with
-    # distributed-ranges/test/gtest/common/counted.cpp:62: Failure Expected
-    # equality of these values: *(--counted_result.end()) Which is: 5, should be
-    # 77 Mdspan, Mdarray hangs sometimes on ISHMEM.
+#Some Halo3 cases don't work on IshmemBackend (work on MPI, see: DRA-84),
+#Counted.large fails with
+#distributed -                                                                 \
+    ranges / test / gtest / common / counted.cpp : 62 : Failure Expected
+#equality of these values : *(--counted_result.end())Which is : 5, should be
+# 77 Mdspan, Mdarray hangs sometimes on ISHMEM.
     set(sycl-exclusions
             ${sycl-exclusions}Halo3/*:Sort*:Counted/*:Mdspan*:Mdarray*:)
   endif()

test/gtest/mp/copy.cpp

@@ -6,9 +6,7 @@
 
 // Fixture
 
-template <typename T> class CopyMP : public testing::Test {
-public:
-};
+template <typename T> class CopyMP : public testing::Test { public: };
 
 TYPED_TEST_SUITE(CopyMP, AllTypes);
 

test/gtest/mp/reduce.cpp

@@ -6,9 +6,7 @@
 
 // Fixture
 
-template <typename T> class ReduceMP : public testing::Test {
-public:
-};
+template <typename T> class ReduceMP : public testing::Test { public: };
 
 TYPED_TEST_SUITE(ReduceMP, AllTypes);
 

test/gtest/mp/segments.cpp

@@ -6,9 +6,7 @@
 
 #include <dr/mp/views/segmented.hpp>
 
-template <typename T> class Segmented : public testing::Test {
-public:
-};
+template <typename T> class Segmented : public testing::Test { public: };
 
 TYPED_TEST_SUITE(Segmented, AllTypesWithoutIshmem);
 
@@ -27,9 +25,7 @@ TYPED_TEST(Segmented, Basic) {
   EXPECT_EQ(dr::ranges::segments(ops.dist_vec), segmented);
 }
 
-template <typename T> class SegmentUtils : public testing::Test {
-public:
-};
+template <typename T> class SegmentUtils : public testing::Test { public: };
 
 // traversing on host over local_segment does not work in case of both:
 // device_memory and IshmemBackend (which uses device memory)

test/gtest/mp/xp-tests.hpp

@@ -20,15 +20,15 @@ namespace xp = dr::mp;
 
 template <typename V>
 concept compliant_view = rng::forward_range<V> && rng::random_access_range<V> &&
-                         rng::viewable_range<V> && requires(V &v) {
-                           // test one at a time so error is apparent
-                           dr::ranges::segments(v);
-                           dr::ranges::segments(v).begin();
-                           *dr::ranges::segments(v).begin();
-                           dr::ranges::rank(*dr::ranges::segments(v).begin());
-                           //  dr::ranges::local(rng::begin(dr::ranges::segments(v)[0]));
-                           //  dr::mp::local_segments(v);
-                         };
+    rng::viewable_range<V> && requires(V &v) {
+  // test one at a time so error is apparent
+  dr::ranges::segments(v);
+  dr::ranges::segments(v).begin();
+  *dr::ranges::segments(v).begin();
+  dr::ranges::rank(*dr::ranges::segments(v).begin());
+  //  dr::ranges::local(rng::begin(dr::ranges::segments(v)[0]));
+  //  dr::mp::local_segments(v);
+};
 
 inline void barrier() { dr::mp::barrier(); }
 inline void fence() { dr::mp::fence(); }