From aa6f0ace2f32985a4f8710ae5881615fdebe35a6 Mon Sep 17 00:00:00 2001
From: Howard Huang <howardhuang@fb.com>
Date: Thu, 9 Feb 2023 10:44:59 -0800
Subject: [PATCH] Remove API declarations in Ops.hpp (#94532)

In #91257, we removed direct calls to methods in ops.cpp, so this is updating to also remove ops.hpp
Pull Request resolved: https://github.com/pytorch/pytorch/pull/94532
Approved by: https://github.com/kwen2501
---
 torch/csrc/distributed/c10d/Ops.cpp |   1 -
 torch/csrc/distributed/c10d/Ops.hpp | 117 ----------------------------
 2 files changed, 118 deletions(-)
 delete mode 100644 torch/csrc/distributed/c10d/Ops.hpp

diff --git a/torch/csrc/distributed/c10d/Ops.cpp b/torch/csrc/distributed/c10d/Ops.cpp
index 4f319f0b221373..b2bd7fe0d42f58 100644
--- a/torch/csrc/distributed/c10d/Ops.cpp
+++ b/torch/csrc/distributed/c10d/Ops.cpp
@@ -1,6 +1,5 @@
 #include <ATen/core/dispatch/Dispatcher.h>
 #include <c10/util/intrusive_ptr.h>
-#include <torch/csrc/distributed/c10d/Ops.hpp>
 #include <torch/csrc/distributed/c10d/ProcessGroup.hpp>
 #include <torch/csrc/distributed/c10d/Types.hpp>
 #include <torch/library.h>
diff --git a/torch/csrc/distributed/c10d/Ops.hpp b/torch/csrc/distributed/c10d/Ops.hpp
deleted file mode 100644
index e414640cccac16..00000000000000
--- a/torch/csrc/distributed/c10d/Ops.hpp
+++ /dev/null
@@ -1,117 +0,0 @@
-#pragma once
-
-#include <c10/util/intrusive_ptr.h>
-#include <torch/csrc/distributed/c10d/ProcessGroup.hpp>
-
-namespace c10d {
-namespace ops {
-
-// Below are essentially ProcessGroup's corresponding ops but routed to the
-// dispatcher. To be noted, it's a convention to use at::TensorList to represent
-// const std::vector<at::Tensor>&. However, const std::vector<at::Tensor>& is
-// used whenever the API accepts std::vector<std::vector<at::Tensor>>& to keep
-// consistency.
-TORCH_API c10::intrusive_ptr<Work> broadcast(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::TensorList tensors,
-    const BroadcastOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> allreduce(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::TensorList tensors,
-    const AllreduceOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> allreduce_coalesced(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::TensorList tensors,
-    const AllreduceCoalescedOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> allgather(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const std::vector<std::vector<at::Tensor>>& output_tensors,
-    at::TensorList input_tensors,
-    const AllgatherOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> _allgather_base(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::Tensor& outputTensor,
-    at::Tensor& inputTensor,
-    const AllgatherOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> allgather_coalesced(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const std::vector<std::vector<at::Tensor>>& output_lists,
-    const at::TensorList& input_list,
-    const AllgatherOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> reduce_scatter(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const at::TensorList& output_tensors,
-    const std::vector<std::vector<at::Tensor>>& input_tensors,
-    const ReduceScatterOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> _reduce_scatter_base(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-     at::Tensor& output_tensor,
-     at::Tensor& input_tensor,
-    const ReduceScatterOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> reduce(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::TensorList tensors,
-    const ReduceOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> gather(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const std::vector<std::vector<at::Tensor>>& output_tensors,
-    const at::TensorList& input_tensors,
-    const GatherOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> scatter(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const at::TensorList& output_tensors,
-    const std::vector<std::vector<at::Tensor>>& input_tensors,
-    const ScatterOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> alltoall_base(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::Tensor& output,
-    at::Tensor& input,
-    const std::vector<int64_t> outputSplitSizes,
-    const std::vector<int64_t> inputSplitSizes,
-    const AllToAllOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> alltoall(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const at::TensorList& output_tensors,
-    const at::TensorList& input_tensors,
-    const AllToAllOptions& opts = {});
-
-TORCH_API c10::intrusive_ptr<Work> barrier(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const BarrierOptions& opts = {});
-
-TORCH_API void monitored_barrier(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    const BarrierOptions& opts,
-    bool waitAllRanks);
-
-TORCH_API c10::intrusive_ptr<Work> send(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::TensorList tensors,
-    int64_t dstRank,
-    int64_t tag);
-
-TORCH_API c10::intrusive_ptr<Work> recv(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::TensorList tensors,
-    int64_t srcRank,
-    int64_t tag);
-
-TORCH_API c10::intrusive_ptr<Work> recv_any_source(
-    const c10::intrusive_ptr<ProcessGroup>& process_group,
-    at::TensorList tensors,
-    int64_t tag);
-
-} // namespace ops
-} // namespace c10d