junrushao1994 edited a comment 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. A4. We will add a lot of comments on how this hash map works. So it requires some amount of work. I will turn it back to draft until the work is done :-) ---------------------------------------------------------------- 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]
