srkreddy1238 commented on issue #4102: URL: https://github.com/apache/tvm/issues/4102#issuecomment-783514261
@wx3000 thanks for the inputs. I see TF2.x as an amendment rather a replacement to 1.x parser. There are new operator support needed in 2.x which can go as an amend over existing operator/parser support. The control flow ops and List ops can be extended on existing parser infra. Ref. #7498 - Extends parser to accept concrete_function of TF2.x from which it extracts graph. - Once we have the graph it's all about operators support. - I can see most of the ops are supported out of the existing parser. Couldn't find a concrete motivation for a brand new parser which overlaps existing parser mostly. @yongwww any thoughts ? ---------------------------------------------------------------- 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]
