fix warning in clang

oneflow/graph/model_save_comp_task_node.h

@@ -21,8 +21,8 @@ class MdSaveCompTaskNode final : public CompTaskNode {
   }
 
  private:
-  void BuildExecAndEnrollLbn2Regsts(TaskGraph* gph);
-  void InferShapeOfBlobsInProducedRegsts(TaskGraph* gph);
+  void BuildExecAndEnrollLbn2Regsts(TaskGraph* gph) override;
+  void InferShapeOfBlobsInProducedRegsts(TaskGraph* gph) override;
   bool IsMeaningLess() const override {
     return !GetSubscribedRegstDesc("model");
   }

oneflow/graph/model_update_comp_task_node.h

@@ -12,8 +12,8 @@ class MdUpdtCompTaskNode final : public CompTaskNode {
   ~MdUpdtCompTaskNode() = default;
 
  private:
-  void BuildExecAndEnrollLbn2Regsts(TaskGraph* gph);
-  void InferShapeOfBlobsInProducedRegsts(TaskGraph* gph);
+  void BuildExecAndEnrollLbn2Regsts(TaskGraph* gph) override;
+  void InferShapeOfBlobsInProducedRegsts(TaskGraph* gph) override;
   TaskType task_type() const override {
     return kMdUpdtCompTask;
   }

oneflow/thread/thread_manager.cpp

@@ -28,17 +28,17 @@ ThreadMgr::ThreadMgr() {
   threads_.reserve(dev_num_per_machine + 3);
   for (uint64_t dev_phy_id = 0; dev_phy_id < dev_num_per_machine; ++dev_phy_id){
     if (device_type == kGPU) {
-      threads_.push_back(std::move(of_make_unique<GpuThread>(dev_phy_id)));
+      threads_.push_back(of_make_unique<GpuThread>(dev_phy_id));
     } else {
-      threads_.push_back(std::move(of_make_unique<CpuThread>()));
+      threads_.push_back(of_make_unique<CpuThread>());
     }
   }
   // cpu thread - for persistence
-  threads_.push_back(std::move(of_make_unique<CpuThread>()));
+  threads_.push_back(of_make_unique<CpuThread>());
   // cpu thread - for boxing
-  threads_.push_back(std::move(of_make_unique<CpuThread>()));
+  threads_.push_back(of_make_unique<CpuThread>());
   // cpu thread - for commnet
-  threads_.push_back(std::move(of_make_unique<CpuThread>()));
+  threads_.push_back(of_make_unique<CpuThread>());
 }
 
 }  // namespace oneflow