[GitHub] [incubator-tvm] tqchen commented on pull request #6779: [WIP] Object Schema

[GitBox <gi...@apache.org>]

tqchen commented on pull request #6779:
URL: https://github.com/apache/incubator-tvm/pull/6779#issuecomment-720692796


   We acknowledged that it is not ideal to elaborate everything in the schema(e.g. describe the member function). So the main goal of the schema serves as a way to facilitate the data layout generation, while still allowing users to fill out the necessary content(e.g. member functions) in the corresponding backend language. 
   
   That means the schema is an auxiliary tool that developer can run, but the generated code is always checked in and generation step is not invoked part of the build (like clang-format), resulting lightweight commitment and transition from developers. 
   
   The layout information alone is very useful for cross language enablement, like rust access binding etc and build code generators for accessing the data structure.
   
   Given that first class python support is one of our design goal, and as we start to evolve TVMScript I think we want to think about the possibility of supporting custom ADT as part of the type system and input. I do think it makes sense to have a python frontend to the schema itself, which provides convenience to specify the data layout.
   
   
   
   


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