You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/06/18 18:11:45 UTC

[GitHub] [incubator-tvm] junrushao1994 edited a comment on pull request #5740: [Object] Introduce POD-C Compliant tvm::Map

junrushao1994 edited a comment on pull request #5740:
URL: https://github.com/apache/incubator-tvm/pull/5740#issuecomment-644613259


   Summary of the change:
   - Remove most of the short names (i, n, m, d, b) and use more informative ones instead.
   - Added comments between important code blocks (insert vs rehash)
   - Add test cases to cover the switch between SmallMapNode and DenseMapNode
   


----------------------------------------------------------------
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