#2063: Update void* to std::byte* in Holder struct

Author: thearusable

src/vt/objgroup/holder/holder.h

@@ -63,7 +63,7 @@ struct Holder final : HolderObjBase<ObjT> {
 public:
   ObjT* get() override { return obj_.get(); }
   bool exists() override { return obj_ != nullptr; }
-  void* getPtr() override { return obj_.get(); }
+  std::byte* getPtr() override { return reinterpret_cast<std::byte*>(obj_.get()); }
 
   template <typename... Args>
   void reset(Args&&... args) {

src/vt/objgroup/holder/holder_base.h

@@ -57,7 +57,7 @@ struct HolderBase {
 
   virtual ~HolderBase() = default;
   virtual bool exists() = 0;
-  virtual void* getPtr() = 0;
+  virtual std::byte* getPtr() = 0;
 
   template <typename Serializer>
   void serialize(Serializer& s) {
@@ -78,7 +78,7 @@ template <typename ObjT>
 struct HolderObjBase : HolderBase {
   virtual ~HolderObjBase() = default;
   virtual ObjT* get() = 0;
-  virtual void* getPtr() = 0;
+  virtual std::byte* getPtr() = 0;
 };
 
 }}} /* end namespace vt::objgroup::holder */

src/vt/objgroup/holder/holder_basic.h

@@ -61,7 +61,7 @@ struct HolderBasic final : HolderObjBase<ObjT> {
 public:
   ObjT* get() override { return obj_; }
   bool exists() override { return obj_ != nullptr; }
-  void* getPtr() override { return obj_; }
+  std::byte* getPtr() override { return reinterpret_cast<std::byte*>(obj_); }
 
   template <typename... Args>
   void reset(Args&&... args) {

src/vt/objgroup/manager.impl.h

@@ -288,7 +288,7 @@ ObjT* ObjGroupManager::get(ProxyElmType<ObjT> proxy) {
   auto iter = objs_.find(proxy_bits);
   vtAssert(iter != objs_.end(), "Obj must exist on this node");
   HolderBaseType* holder = iter->second.get();
-  return static_cast<ObjT*>(holder->getPtr());
+  return reinterpret_cast<ObjT*>(holder->getPtr());
 }
 
 template <typename ObjT, typename... Args>

src/vt/objgroup/manager.static.h

@@ -96,7 +96,7 @@ decltype(auto) invoke(
   return runnable::makeRunnableVoid(false, han, this_node)
     .withObjGroup(elm)
     .withLBData(lb_data, elm_id)
-    .runLambda(f, static_cast<ObjT*>(elm), msg.get());
+    .runLambda(f, reinterpret_cast<ObjT*>(elm), msg.get());
 }
 
 template <typename MsgT>

src/vt/objgroup/proxy/proxy_objgroup.impl.h

@@ -279,7 +279,7 @@ typename Proxy<ObjT>::PendingSendType Proxy<ObjT>::reduce(MsgPtrT inmsg, ReduceS
 template <typename ObjT>
 ObjT* Proxy<ObjT>::get() const {
   auto proxy = Proxy<ObjT>(*this);
-  return theObjGroup()->get<ObjT>(proxy);
+  return reinterpret_cast<ObjT*>(theObjGroup()->get<ObjT>(proxy));
 }
 
 template <typename ObjT>