masahi commented on a change in pull request #6906:
URL: https://github.com/apache/incubator-tvm/pull/6906#discussion_r522514891
##########
File path: src/relay/op/tensor/transform.cc
##########
@@ -1076,7 +1076,7 @@ Examples::
.set_support_level(3)
.add_type_rel("Take", TakeRel)
.set_attr<FTVMCompute>("FTVMCompute", TakeCompute)
- .set_attr<TOpPattern>("TOpPattern", kInjective);
+ .set_attr<TOpPattern>("TOpPattern", kOpaque);
Review comment:
Yes a new pass that inserts `stop_fusion` sounds good. I can work on
this. We can make `take` opaque for now until I have that new pass ready.
Anyway I think graph runtime shouldn't be affected by the issue of `take` +
dynamic, so `take` should be injective eventually.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
