From 349a9e46578a3862d5b71cfd25e86948d595e328 Mon Sep 17 00:00:00 2001 From: chenyemeng Date: Sat, 28 Nov 2020 13:46:03 +0800 Subject: [PATCH] modify comments --- ge/graph/manager/graph_manager.cc | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ge/graph/manager/graph_manager.cc b/ge/graph/manager/graph_manager.cc index f00c477f..d14d691b 100755 --- a/ge/graph/manager/graph_manager.cc +++ b/ge/graph/manager/graph_manager.cc @@ -2323,11 +2323,12 @@ Status GraphManager::OptimizeStage2(ge::ComputeGraphPtr &compute_graph) { GE_CHK_STATUS_RET(pass_for_control_attr_optimize.AddPass("OptimizeStage2::AfterMergePasses::" "EndOfSequenceAddControlPass", new (std::nothrow) EndOfSequenceAddControlPass)) - // SubgraphPass solves memory_assign_conflicts by insert MemcpyAsync node, which depends on multi attrs and - // graph-structure. So try not to add new pass after SubgraphPass. + // 'SubgraphPass' solves memory_assign_conflicts by insert MemcpyAsync node, which depends on multi attrs and + // graph-structure. Passes after 'SubgraphPass' MUST NOT remove MemcpyAsync/Identity nodes in subgraphs. GE_CHK_STATUS_RET(pass_for_control_attr_optimize.AddPass("OptimizeStage2::ControlAttrOptimize::SubgraphPass", new (std::nothrow) SubgraphPass)) - // AttachStreamLabelPass modifies attr without changing structure of compute_graph + // 'AttachStreamLabelPass' modifies attr without changing structure of compute_graph + // All passes after 'AttachStreamLabelPass' MUST mark stream_label on new nodes by self. GE_CHK_STATUS_RET(pass_for_control_attr_optimize.AddPass("OptimizeStage2::ControlAttrOptimize::AttachStreamLabelPass", new (std::nothrow) AttachStreamLabelPass))