From 18505a8af8c6ec22a06946afed3214781b10610e Mon Sep 17 00:00:00 2001 From: Bradley Dice Date: Wed, 4 Dec 2024 10:17:21 -0600 Subject: [PATCH] Fix CCCL 2.7.0-rc2 compile issue by removing reference from values. (#4799) We are planning to migrate to CCCL 2.7.0 in RAPIDS 25.02. I found a compilation error in `libcugraph_etl` that is fixed by removing a reference `&` from a device comparator, and passing the parameters by value instead. This is a pretty small fix but I expect we will want to build 24.12 with newer CCCL versions for DLFW releases. To avoid needing a backport for DLFW, I am targeting 24.12 with this change. I can retarget to 25.02 if desired. This is extracted from #4743, but that PR shouldn't be merged (it includes testing changes that are not intended for merge). Authors: - Bradley Dice (https://github.com/bdice) Approvers: - Chuck Hastings (https://github.com/ChuckHastings) URL: https://github.com/rapidsai/cugraph/pull/4799 --- cpp/libcugraph_etl/src/renumbering.cu | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cpp/libcugraph_etl/src/renumbering.cu b/cpp/libcugraph_etl/src/renumbering.cu index a2a20d5b2dc..0d47fad8000 100644 --- a/cpp/libcugraph_etl/src/renumbering.cu +++ b/cpp/libcugraph_etl/src/renumbering.cu @@ -730,7 +730,7 @@ __global__ static void select_unrenumber_string(str_hash_value* idx_to_col_row, } struct struct_sort_descending { - __host__ __device__ bool operator()(str_hash_value& a, str_hash_value& b) + __host__ __device__ bool operator()(str_hash_value a, str_hash_value b) { return (a.count_ > b.count_); }