#2281: use or remove unused variables

Author: cwschilly

src/vt/collective/reduce/allreduce/rabenseifner.impl.h

@@ -93,7 +93,6 @@ void Rabenseifner::storeData(size_t id, Args&&... data) {
   }
 
   state.local_col_wait_count_++;
-  auto const is_ready = state.local_col_wait_count_ == local_num_elems_;
 }
 
 template <typename DataT, template <typename Arg> class Op>

src/vt/objgroup/manager.impl.h

@@ -289,7 +289,6 @@ ObjGroupManager::allreduce(ProxyType<ObjT> proxy, Args&&... data) {
   // We only support allreduce with a single data type
   using DataT = typename std::tuple_element<0, typename Trait::TupleType>::type;
 
-  auto const this_node = vt::theContext()->getNode();
   auto const strong_proxy = vt::collective::reduce::detail::StrongObjGroup{proxy.getProxy()};
   auto const id = StateHolder::getNextID(strong_proxy);
 

tests/perf/allreduce.cc

@@ -71,7 +71,10 @@ static constexpr std::array<size_t, 9> const payloadSizes = {
   64, 128, 2048, 16384, 32768, 524288, 1048576, 2097152, 4194304};
 
 struct MyTest : PerfTestHarness {
-  MyTest() { DisableGlobalTimer(); }
+  MyTest() {
+    DisableGlobalTimer();
+    data.reserve(*std::max_element(payloadSizes.begin(), payloadSizes.end()));
+  }
 
   std::vector<int32_t> data;
 };