Browse Source

!1105 Bugfix: add remove reformat for unknonwn shape

From: @wan_xuelei
Reviewed-by: @xchu42,@ji_chen
Signed-off-by: @ji_chen
tags/v1.2.0
mindspore-ci-bot Gitee 3 years ago
parent
commit
1d354d1e29
1 changed files with 31 additions and 10 deletions
  1. +31
    -10
      ge/graph/passes/reshape_remove_pass.cc

+ 31
- 10
ge/graph/passes/reshape_remove_pass.cc View File

@@ -15,29 +15,50 @@
*/ */


#include "graph/passes/reshape_remove_pass.h" #include "graph/passes/reshape_remove_pass.h"

#include <map>
#include <string>

#include "framework/common/util.h" #include "framework/common/util.h"
#include "framework/common/types.h"
#include "graph/passes/pass_utils.h" #include "graph/passes/pass_utils.h"
#include "graph/utils/node_utils.h" #include "graph/utils/node_utils.h"


namespace ge { namespace ge {
namespace { namespace {
const int kReshapeDataIndex = 0; const int kReshapeDataIndex = 0;
enum OpHashValue {
kReshapeType = 0,
kReformatType = 1,
kOpNoDelete = -1
};

std::map<std::string, OpHashValue> kToBeDeleteOp = {
{RESHAPE, kReshapeType},
{REFORMAT, kReformatType}
};
} }


Status ReshapeRemovePass::Run(NodePtr &node) { Status ReshapeRemovePass::Run(NodePtr &node) {
GE_CHECK_NOTNULL(node); GE_CHECK_NOTNULL(node);
GE_CHECK_NOTNULL(node->GetOpDesc()); GE_CHECK_NOTNULL(node->GetOpDesc());
if (node->GetType() != RESHAPE && node->GetType() != REFORMAT) {
return SUCCESS;
}

bool is_shape_unknown = false;
if (NodeUtils::GetNodeUnknownShapeStatus(*node, is_shape_unknown) == GRAPH_SUCCESS) {
if (is_shape_unknown) {
GELOGI("op:%s is unknown shape, can not be deleted.",
node->GetName().c_str());
return SUCCESS;
int key = kToBeDeleteOp.find(node->GetType()) == kToBeDeleteOp.end() ? kOpNoDelete : kToBeDeleteOp[node->GetType()];
switch(key) {
case kReshapeType: {
bool is_shape_unknown = false;
if (NodeUtils::GetNodeUnknownShapeStatus(*node, is_shape_unknown) == GRAPH_SUCCESS) {
if (is_shape_unknown) {
GELOGI("op:%s is unknown shape, can not be deleted.",
node->GetName().c_str());
return SUCCESS;
}
}
break;
} }
case kReformatType:
break;
default:
return SUCCESS;
} }


GELOGI("Remove %s node %s", node->GetType().c_str(), node->GetName().c_str()); GELOGI("Remove %s node %s", node->GetType().c_str(), node->GetName().c_str());


Loading…
Cancel
Save