!780 回退 'Pull Request !770 : AssignMemory2HasRefAttrNode'

From: @wqtshg
Reviewed-by: @xchu42,@ji_chen
Signed-off-by: @ji_chen

ge/graph/build/memory/graph_mem_assigner.cc

@@ -89,14 +89,6 @@ Status VariableMemoryAssigner::AssignVarAttr2Nodes() {
   return ge::SUCCESS;
 }
 
-Status VariableMemoryAssigner::AssignMemory2HasRefAttrNode() {
-  Status result = ge::VarMemAssignUtil::AssignMemory2HasRefAttrNode(compute_graph_);
-  if (result != ge::SUCCESS) {
-    return result;
-  }
-  return ge::SUCCESS;
-}
-
 Status GraphMemoryAssigner::AssignMemory() {
   ge::HybridMemAssignerPtr mem_assigner(new (std::nothrow) HybridMemAssigner(compute_graph_));
   if (mem_assigner->Assign() != ge::SUCCESS) {
@@ -139,19 +131,6 @@ ge::Status GraphMemoryAssigner::AssignVarAttr2Nodes() {
   return ge::SUCCESS;
 }
 
-ge::Status GraphMemoryAssigner::AssignMemory2HasRefAttrNode() {
-  auto variable_assigner =
-    std::unique_ptr<ge::VariableMemoryAssigner>(new (std::nothrow) ge::VariableMemoryAssigner(compute_graph_));
-  if (variable_assigner == nullptr) {
-    GELOGE(ge::FAILED, "Alloc VariableMemoryAssigner failed.");
-    return ge::FAILED;
-  }
-  if (variable_assigner->AssignMemory2HasRefAttrNode() != ge::SUCCESS) {
-    return ge::FAILED;
-  }
-  return ge::SUCCESS;
-}
-
 ge::Status GraphMemoryAssigner::CalculateTensorRealSizeAndOutSize(const ge::ConstGeTensorDescPtr &output_desc,
                                                                   int64_t dim_index, int64_t &output_mem_size,
                                                                   int64_t &batch_dim_num, int64_t &out_size) {

ge/graph/build/memory/graph_mem_assigner.h

@@ -63,8 +63,6 @@ class VariableMemoryAssigner {
   ///
   ge::Status AssignVarAttr2Nodes();
 
-  ge::Status AssignMemory2HasRefAttrNode();
-
  private:
   ge::ComputeGraphPtr compute_graph_;
 };
@@ -99,8 +97,6 @@ class GraphMemoryAssigner {
   ///
   ge::Status AssignVarAttr2Nodes();
 
-  ge::Status AssignMemory2HasRefAttrNode();
-
   ge::Status ReAssignMemory(bool is_loop_graph, size_t &mem_offset);
 
   ge::Status AssignZeroCopyMemory(size_t &mem_offset, size_t &zero_mem_copy_size);

ge/graph/build/memory/memory_assigner.cc

@@ -40,11 +40,6 @@ Status MemoryAssigner::AssignMemory(bool is_loop_graph, size_t &mem_offset, size
     return ge::FAILED;
   }
 
-  if (graph_mem_assigner.AssignMemory2HasRefAttrNode() != ge::SUCCESS) {
-    GELOGE(ge::FAILED, "Assign memory 2 HasRef AttrNode failed");
-    return ge::FAILED;
-  }
-
   // Assign memory for reference
   if (graph_mem_assigner.AssignReferenceMemory() != ge::SUCCESS) {
     GELOGE(ge::FAILED, "Assign reference memory failed!");

ge/graph/build/memory/var_mem_assign_util.cc

@@ -34,6 +34,7 @@ using std::vector;
 namespace ge {
 Status VarMemAssignUtil::AssignVarMemory(ge::ComputeGraphPtr &compute_graph) {
   GE_CHK_STATUS_RET(AssignMemory2VariableNode(compute_graph));
+  GE_CHK_STATUS_RET(AssignMemory2HasRefAttrNode(compute_graph));
 
   return SUCCESS;
 }