huajsj edited a comment on issue #4703: [VTA] Support network which have no 
unique operator as start/stop name for graph pack.
URL: https://github.com/apache/incubator-tvm/pull/4703#issuecomment-575405296
 
 
   > I suggest we can follow a two-pass approach. For the first pass, get a 
list of named operator occurrence; then locate the subgraph by counting 
operator occurrence.
   
   Hi @liangfu , I try this feature couple times, and feel the way to count 
operator occurrence times is not very convenience,  and using the index 
directly is more straight forward and efficiency, the two-pass way may work but 
that add additional logic and make the solution too heavy,  I would recommend 
to keep current one pass logic.
   
   Regards
   Hua   
   

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to