[GitHub] [tvm] junrushao1994 edited a comment on pull request #7534: [Target] Add support for target object with host field compatible with previous api

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

junrushao1994 edited a comment on pull request #7534:
URL: https://github.com/apache/tvm/pull/7534#issuecomment-793062861


   Notes on the tutorial/tests change:
   
   As we are changing user-facing tutorials, we should stick to what is the most readable to new-comers. My proposal is:
   
   Replacing:
   
   ```python
   target = "cuda"
   target_host = "llvm"
   ```
   
   with
   
   ```python
   target = tvm.target.Target("cuda", host="llvm")
   ```
   
   This way, we are not deprecating the old ways of using target and target_host, but encouraging the new ways of combining them from the very beginning


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