Address review comments: clang format and ckProfiler typo

Author: EnricoDeg

profiler/include/profiler/profile_gemm_blockscale_wp_impl.hpp

@@ -69,19 +69,19 @@ template <typename A0DataType,
           typename ALayout,
           typename BLayout,
           typename ELayout>
-bool profile_gemm_blockscale_weighpreshuffle_impl(int do_verification,
-                                                  int init_method,
-                                                  bool do_log,
-                                                  bool time_kernel,
-                                                  int M,
-                                                  int N,
-                                                  int K,
-                                                  int StrideA,
-                                                  int StrideB,
-                                                  int StrideE,
-                                                  int n_warmup,
-                                                  int n_iter,
-                                                  uint64_t rotating = 0)
+bool profile_gemm_blockscale_weightpreshuffle_impl(int do_verification,
+                                                   int init_method,
+                                                   bool do_log,
+                                                   bool time_kernel,
+                                                   int M,
+                                                   int N,
+                                                   int K,
+                                                   int StrideA,
+                                                   int StrideB,
+                                                   int StrideE,
+                                                   int n_warmup,
+                                                   int n_iter,
+                                                   uint64_t rotating = 0)
 {
     bool pass = true;
 

profiler/src/profile_gemm_blockscale_wp.cpp

@@ -126,19 +126,19 @@ int profile_gemm_blockscale_weighpreshuffle(int argc, char* argv[])
         const int DefaultStrideB = ck::is_same_v<BLayout, Row> ? N : K;
         const int DefaultStrideE = ck::is_same_v<ELayout, Row> ? N : M;
 
-        bool pass = ck::profiler::profile_gemm_blockscale_weighpreshuffle_impl<A0DataType,
-                                                                               A1DataType,
-                                                                               B0DataType,
-                                                                               B1DataType,
-                                                                               ComputeDataType,
-                                                                               AccDataType,
-                                                                               EDataType,
-                                                                               scale_block_m,
-                                                                               scale_block_n,
-                                                                               scale_block_k,
-                                                                               ALayout,
-                                                                               BLayout,
-                                                                               ELayout>(
+        bool pass = ck::profiler::profile_gemm_blockscale_weightpreshuffle_impl<A0DataType,
+                                                                                A1DataType,
+                                                                                B0DataType,
+                                                                                B1DataType,
+                                                                                ComputeDataType,
+                                                                                AccDataType,
+                                                                                EDataType,
+                                                                                scale_block_m,
+                                                                                scale_block_n,
+                                                                                scale_block_k,
+                                                                                ALayout,
+                                                                                BLayout,
+                                                                                ELayout>(
             do_verification,
             init_method,
             do_log,

test/gemm_blockscale_wp/test_gemm_blockscale_wp_xdl_fp8.cpp

@@ -39,6 +39,7 @@ using KernelTypes_MK_NK = ::testing::Types<
     std::tuple< F8, F32, F8, F32, F8, BF16>
 #endif
     >;
+// clang-format on
 
 TYPED_TEST_SUITE(TestGemmBlockScaleWP_FP8_MK_NK, KernelTypes_MK_NK);
 

test/gemm_blockscale_wp/test_gemm_common.hpp

@@ -44,30 +44,30 @@ class TestGemmBlockscaleWPCommon : public ::testing::Test
 
         all_success =
             all_success &
-            ck::profiler::profile_gemm_blockscale_weighpreshuffle_impl<A0DataType,
-                                                                       A1DataType,
-                                                                       B0DataType,
-                                                                       B1DataType,
-                                                                       ComputeDataType,
-                                                                       F32,
-                                                                       CDataType,
-                                                                       ScaleBlockM,
-                                                                       ScaleBlockN,
-                                                                       ScaleBlockK,
-                                                                       ALayout,
-                                                                       BLayout,
-                                                                       CLayout>(verify_,
-                                                                                init_method_,
-                                                                                log_,
-                                                                                bench_,
-                                                                                M,
-                                                                                N,
-                                                                                K,
-                                                                                StrideA,
-                                                                                StrideB,
-                                                                                StrideC,
-                                                                                n_warmup,
-                                                                                n_iter);
+            ck::profiler::profile_gemm_blockscale_weightpreshuffle_impl<A0DataType,
+                                                                        A1DataType,
+                                                                        B0DataType,
+                                                                        B1DataType,
+                                                                        ComputeDataType,
+                                                                        F32,
+                                                                        CDataType,
+                                                                        ScaleBlockM,
+                                                                        ScaleBlockN,
+                                                                        ScaleBlockK,
+                                                                        ALayout,
+                                                                        BLayout,
+                                                                        CLayout>(verify_,
+                                                                                 init_method_,
+                                                                                 log_,
+                                                                                 bench_,
+                                                                                 M,
+                                                                                 N,
+                                                                                 K,
+                                                                                 StrideA,
+                                                                                 StrideB,
+                                                                                 StrideC,
+                                                                                 n_warmup,
+                                                                                 n_iter);
 
         EXPECT_TRUE(all_success);
     }

test/gemm_multiply_multiply_wp/test_gemm_multiply_multiply_wp_xdl_fp8.cpp

@@ -42,6 +42,7 @@ using KernelTypes_MK_NK = ::testing::Types<
     std::tuple< F8, F8, F8, F32, F32, BF16>
 #endif
     >;
+// clang-format on
 
 TYPED_TEST_SUITE(TestGemmMultiplyMultiplyWP_FP8_MK_NK, KernelTypes_MK_NK);
 

test/gemm_universal_preshuffle/test_gemm_universal_preshuffle_xdl_fp8.cpp

@@ -42,6 +42,7 @@ using KernelTypes_MK_NK = ::testing::Types<
     std::tuple< F8, F8, F8, BF16>
 #endif
     >;
+// clang-format on
 
 TYPED_TEST_SUITE(TestGemmUniversalPreshuffle_FP8_MK_NK, KernelTypes_MK_NK);