From 940e075941e0befe31e78a6519b793ef82516b8d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ra=C3=BAl=20Mar=C3=ADn?= Date: Sat, 11 Sep 2021 22:36:22 +0200 Subject: [PATCH] Remove notNull changes --- src/AggregateFunctions/AggregateFunctionSum.h | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/src/AggregateFunctions/AggregateFunctionSum.h b/src/AggregateFunctions/AggregateFunctionSum.h index f967aded1fa..c28096badfe 100644 --- a/src/AggregateFunctions/AggregateFunctionSum.h +++ b/src/AggregateFunctions/AggregateFunctionSum.h @@ -68,17 +68,6 @@ struct AggregateFunctionSumData if constexpr (std::is_floating_point_v) { -#if defined(__clang__) -/// With clang we can tell the compiler to vectorize and it should work with all types when possible, including floating types -/// Something around the number of SSE registers * the number of elements fit in register. -/// Recommend half of those for each vectorization -#pragma clang loop vectorize(enable) vectorize_width(64 / sizeof(T)) unroll_count(128 / sizeof(T)) - for (size_t i = 0; i < count; i++) - { - Impl::add(sum, ptr[i]); - } - return; -#endif /// Compiler cannot unroll this loop, do it manually. /// (at least for floats, most likely due to the lack of -fassociative-math)