samskalicky commented on pull request #18904: URL: https://github.com/apache/incubator-mxnet/pull/18904#issuecomment-680301323
> @samskalicky, can clarify which symbols are needed for the external operators you are interested in? Not sufficiently, anything that anybody uses to write an internal/backend operator could be anything in MXNet/NNVM/TVM/mshadow/etc... so if the goal of the feature is enable any backend operator to be dynamically loaded, we have to export all symbols. The biggest problem with individual symbol exposure many symbols needed arent entirely defined in MXNet source code. One example is that the operator registration macro `NNVM_REGISTER_OP` resolves to something that calls `::dmlc::Registry<::nnvm::Op>::Get()` which is defined in **3rdparty/tvm/nnvm/src/core/op.cc**. So unless we go and modify that 3rd party code we're stuck. ---------------------------------------------------------------- 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]
