thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-18 04:12:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #16343 from ClickHouse/fix-perf-test-functions-logical

Browse Source 
Fix performance test "functions logical" after move to clang
This commit is contained in:
alexey-milovidov 2020-10-25 04:04:24 +03:00 committed by GitHub
commit abb9b25fa0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Functions/FunctionsLogical.cpp
View File

@ -290,10 +290,9 @@ private:
/// Apply target function by feeding it "batches" of N columns /// Apply target function by feeding it "batches" of N columns
/// Combining 10 columns per pass is the fastest for large columns sizes. /// Combining 8 columns per pass is the fastest method, because it's the maximum when clang vectorizes a loop.
/// For small columns sizes - more columns is faster.
template < template <
typename Op, template <typename, size_t> typename OperationApplierImpl, size_t N = 10> typename Op, template <typename, size_t> typename OperationApplierImpl, size_t N = 8>
struct OperationApplier struct OperationApplier
{ {
template <typename Columns, typename ResultData> template <typename Columns, typename ResultData>