zxybazh commented on pull request #7534:
URL: https://github.com/apache/tvm/pull/7534#issuecomment-793079051
> Notes on the tutorial/tests change:
>
> As we are changing user-facing tutorials, we should stick to what is the
most readable & encouraged way of using Target APIs 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` separately, but encouraging the new ways of combining them from
the very beginning for a new-comer, which could make life easier if we finally
agree to deprecate old APIs.
Good point. I have fixed that in the tutorials and another way of declaring
targets like `Target(
"cuda --host=llvm")` was also demonstrated.
----------------------------------------------------------------
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]
