From 92d874be50fd8a8b7fda8b865ff8499dccbca152 Mon Sep 17 00:00:00 2001
From: Richard Barnes <rbarnes@umn.edu>
Date: Mon, 29 Jul 2024 11:11:24 -0400
Subject: [PATCH] Remove unused but set variable from all_reduce.h (#1258)

Allows `-Wunused-but-set-variable` to pass

[ROCm/rccl commit: d09b152aa098dc820898ade4d159687886b3f507]
---
 projects/rccl/src/device/all_reduce.h | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/projects/rccl/src/device/all_reduce.h b/projects/rccl/src/device/all_reduce.h
index 39e7d3d62b..1d431bce23 100644
--- a/projects/rccl/src/device/all_reduce.h
+++ b/projects/rccl/src/device/all_reduce.h
@@ -59,14 +59,6 @@ namespace {
     }
 #endif
 
-    int minChunkSize;
-    if (Proto::Id == NCCL_PROTO_LL)
-      minChunkSize = nthreads*(Proto::calcBytePerGrain()/sizeof(T));
-    if (Proto::Id == NCCL_PROTO_LL128) {
-      // We should not need the final /2 but it makes performance much, much smoother. Might be a bug somewhere.
-      minChunkSize = nthreads*(Proto::calcBytePerGrain()/sizeof(T))/2;
-    }
-
     Primitives<T, RedOp, FanSymmetric<1>, 0, Proto, 0> prims
       (tid, nthreads, &ring->prev, &ring->next, args->sendbuff, args->recvbuff, args->redOpArg, 0, args->connIndex, args->connIndex);
 
@@ -1030,4 +1022,4 @@ struct RunWorkElement<ncclFuncAllReduce, T, RedOp, NCCL_ALGO_TREE, NCCL_PROTO_LL
     runTreeSplit<T, RedOp, ProtoLL128>(args);
     //LAUNCH_CLIQUE_KERNEL(AllReduceCliqueSplitKernel, RedOp, T, args);
   }
-};
\ No newline at end of file
+};