Fix ProcessGroupXCCL options initialization (#2203)

Group UID needs to be set before creating the LogPrefix, and the
initialization list order is fixed now.

In addition, we remove the redefinition of certain members in
ProcessGroupXCCL.hpp which then allows the existing python bindings to
work properly.

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: frost-intel

src/xccl/ProcessGroupXCCL.cpp

@@ -369,15 +369,15 @@ ProcessGroupXCCL::ProcessGroupXCCL(
     int size,
     c10::intrusive_ptr<Options> options)
     : Backend(rank, size),
-      store_(store),
-      options_(std::move(options)),
+      store_(std::move(store)),
       xcclCommCounter_(0),
-      local_id_(process_group_id++) {
+      local_id_(process_group_id++),
+      options_(std::move(options)) {
+  this->setGroupUid(options_->group_name);
   logPrefix_ = createLogPrefix();
   blockingWait_ = getCvarBool(TORCH_XCCL_BLOCKING_WAIT, false);
   traceBufferSize_ = getCvarInt({"TORCH_FR_BUFFER_SIZE"}, 2000);
 
-  this->setGroupUid(options_->group_name);
   // In PGNCCL, the pg ranks are recorded on comm setup in each op, but we just
   // do it here.
   const auto XcclVersion = getXcclVersion();

src/xccl/ProcessGroupXCCL.hpp

@@ -128,8 +128,6 @@ class TORCH_API ProcessGroupXCCL : public Backend {
       return c10::make_intrusive<Options>(is_high_priority_stream);
     }
     bool is_high_priority_stream;
-    std::vector<uint64_t> global_ranks_in_group;
-    std::string group_name;
   };
 
   ProcessGroupXCCL(