zxy844288792 commented on a change in pull request #8464:
URL: https://github.com/apache/tvm/pull/8464#discussion_r669168791
##########
File path: src/relay/transforms/partition_graph.cc
##########
@@ -301,7 +302,7 @@ class Partitioner : public MixedModeMutator {
}
std::string target = end_node->attrs.as<CompilerAttrs>()->compiler;
- std::string name = target + "_" + std::to_string(region->GetID());
+ std::string name = target + "_" + std::to_string(region->GetID()) +
region->GetName();
Review comment:
Agree, the only reason I put in that way is due to when name is
"dafault" then GetName() method will return an empty string and I was afraid it
would cause some issues if i changed that. I agree this is more clear and will
test it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
