From e16ed8102a7f2536d03c014b7cfb7078b1a8df8c Mon Sep 17 00:00:00 2001 From: Martin Marenz Date: Wed, 23 Aug 2023 18:16:33 +0200 Subject: [PATCH] Fixed processed bytes calculation in APPLY_BOOLEAN_MASK benchmark. (#13937) Due to missing parentheses in APPLY_BOOLEAN_MASK benchmark, the number of written bytes were not multiplied by the number of iterations of this benchmark. This patch relates to #13735. Authors: - Martin Marenz (https://github.com/Blonck) Approvers: - Nghia Truong (https://github.com/ttnghia) - David Wendt (https://github.com/davidwendt) URL: https://github.com/rapidsai/cudf/pull/13937 --- cpp/benchmarks/stream_compaction/apply_boolean_mask.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cpp/benchmarks/stream_compaction/apply_boolean_mask.cpp b/cpp/benchmarks/stream_compaction/apply_boolean_mask.cpp index f352d031240..a6feaf04842 100644 --- a/cpp/benchmarks/stream_compaction/apply_boolean_mask.cpp +++ b/cpp/benchmarks/stream_compaction/apply_boolean_mask.cpp @@ -65,7 +65,7 @@ void calculate_bandwidth(benchmark::State& state, cudf::size_type num_columns) (column_bytes_out + validity_bytes_out) * num_columns; // writing columns state.SetItemsProcessed(state.iterations() * column_size * num_columns); - state.SetBytesProcessed(static_cast(state.iterations()) * bytes_read + bytes_written); + state.SetBytesProcessed(static_cast(state.iterations()) * (bytes_read + bytes_written)); } } // namespace