use & for map clear

pull/5353/head
simson 5 years ago
parent d4babf190c
commit 9e94d7df5c

@ -1261,11 +1261,11 @@ void PynativeExecutor::GradNetInner(const GradOperationPtr &grad, const py::obje
} }
template <typename T> template <typename T>
void MapClear(T map, const std::string &flag) { void MapClear(T *map, const std::string &flag) {
for (auto it = map.begin(); it != map.end();) { for (auto it = map->begin(); it != map->end();) {
if (it->first.find(flag) != std::string::npos) { if (it->first.find(flag) != std::string::npos) {
it->second = nullptr; it->second = nullptr;
it = map.erase(it); it = map->erase(it);
} else { } else {
it++; it++;
} }
@ -1275,9 +1275,9 @@ void MapClear(T map, const std::string &flag) {
void PynativeExecutor::Clear(const std::string &flag) { void PynativeExecutor::Clear(const std::string &flag) {
if (!flag.empty()) { if (!flag.empty()) {
MS_LOG(DEBUG) << "Clear res"; MS_LOG(DEBUG) << "Clear res";
MapClear<std::unordered_map<std::string, FuncGraphPtr>>(graph_map_, flag); MapClear<std::unordered_map<std::string, FuncGraphPtr>>(&graph_map_, flag);
MapClear<std::unordered_map<std::string, FuncGraphPtr>>(cell_graph_map_, flag); MapClear<std::unordered_map<std::string, FuncGraphPtr>>(&cell_graph_map_, flag);
MapClear<std::unordered_map<std::string, ResourcePtr>>(cell_resource_map_, flag); MapClear<std::unordered_map<std::string, ResourcePtr>>(&cell_resource_map_, flag);
Clean(); Clean();
// Maybe exit in the pynative runing op, so need reset pynative flag. // Maybe exit in the pynative runing op, so need reset pynative flag.
auto ms_context = MsContext::GetInstance(); auto ms_context = MsContext::GetInstance();
@ -1309,17 +1309,17 @@ void PynativeExecutor::Clean() {
} }
template <typename T> template <typename T>
void MapErase(T map) { void MapErase(T *map) {
for (auto it = map.begin(); it != map.end();) { for (auto it = map->begin(); it != map->end();) {
it = map.erase(it++); it = map->erase(it++);
} }
} }
void PynativeExecutor::ClearRes() { void PynativeExecutor::ClearRes() {
MapErase<std::unordered_map<std::string, FuncGraphPtr>>(graph_map_); MapErase<std::unordered_map<std::string, FuncGraphPtr>>(&graph_map_);
MapErase<std::unordered_map<std::string, FuncGraphPtr>>(cell_graph_map_); MapErase<std::unordered_map<std::string, FuncGraphPtr>>(&cell_graph_map_);
MapErase<std::unordered_map<std::string, ResourcePtr>>(cell_resource_map_); MapErase<std::unordered_map<std::string, ResourcePtr>>(&cell_resource_map_);
MapErase<std::unordered_map<std::string, abstract::AbstractBasePtr>>(node_abs_map_); MapErase<std::unordered_map<std::string, abstract::AbstractBasePtr>>(&node_abs_map_);
Clean(); Clean();
resource_.reset(); resource_.reset();
} }

Loading…
Cancel
Save