fmt

Author: AntoinePrv

cpp/src/arrow/util/bpacking.cc

@@ -32,11 +32,11 @@ struct UnpackDynamicFunction {
   using Implementation = std::pair<DispatchLevel, FunctionType>;
 
   static constexpr auto implementations() {
-    return std::array {
+    return std::array{
 #if defined(ARROW_HAVE_SSE4_2)
-      Implementation{DispatchLevel::NONE, &unpack_sse4_2<Uint>},
+        Implementation{DispatchLevel::NONE, &unpack_sse4_2<Uint>},
 #else
-      Implementation{DispatchLevel::NONE, &unpack_scalar<Uint>},
+        Implementation{DispatchLevel::NONE, &unpack_scalar<Uint>},
 #endif
 #if defined(ARROW_HAVE_RUNTIME_AVX2)
         Implementation{DispatchLevel::AVX2, &unpack_avx2<Uint>},

cpp/src/arrow/util/bpacking_simd_kernel_internal.h

@@ -842,16 +842,15 @@ struct ForwardToKernel : WorkingKernel {
 // Benchmarking show unpack to uint64_t is underperforming on SSE4.2 and Avx2
 template <typename Traits, typename Arch = typename Traits::arch_type>
 constexpr bool MediumShouldUseUint32 =
-    (HasSse2<Arch> || HasSse2<Arch>)&&  //
+    (HasSse2<Arch> || HasSse2<Arch>) &&  //
     (Traits::kShape.unpacked_byte_size() == sizeof(uint64_t)) &&
     (Traits::kShape.packed_bit_size() < 32) &&
     KernelTraitsWithUnpack<Traits, uint32_t>::kShape.is_medium();
 
 // Benchmarking show large unpack to uint8_t is underperforming on SSE4.2
 template <typename Traits, typename Arch = typename Traits::arch_type>
-constexpr bool LargeShouldUseUint16 = HasSse2<Arch> &&
-                                      (Traits::kShape.unpacked_byte_size() ==
-                                       sizeof(uint8_t));
+constexpr bool LargeShouldUseUint16 =
+    HasSse2<Arch> && (Traits::kShape.unpacked_byte_size() == sizeof(uint8_t));
 
 // A ``std::enable_if`` that works on MSVC
 template <typename Traits>