junrushao1994 commented on pull request #5740: URL: https://github.com/apache/incubator-tvm/pull/5740#issuecomment-642286757
Some actionable items per offline discussion with @tqchen: A1. We move things back to include/tvm/node, instead of runtime. This is mainly due to the concern of keeping tvm runtime minimal. Once we find a strong use case that really need HashMap to exist in tvm runtime, we can bring it back. A2. We will implement a specialization for small maps due to the concern of memory overhead. A3. We will have unordered_map as a backup option, which can be turned on using macro or something like that. ---------------------------------------------------------------- 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]
