Skip to content

[Target] Add cuda tags and unit test#7410

Merged
comaniac merged 6 commits into
apache:mainfrom
zxybazh:main
Feb 5, 2021
Merged

[Target] Add cuda tags and unit test#7410
comaniac merged 6 commits into
apache:mainfrom
zxybazh:main

Conversation

@zxybazh

@zxybazh zxybazh commented Feb 4, 2021

Copy link
Copy Markdown
Member

This commit added cuda tags of most cuda devices for target specification and completed cuda target tag attributes.

@zxybazh

zxybazh commented Feb 4, 2021

Copy link
Copy Markdown
Member Author

@junrushao1994 Please review, thanks!

@junrushao junrushao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall it looks to me. Please see my only comment below. Let's get it in quickly after fixing the linter

Comment thread src/target/tag.cc Outdated
Comment thread src/target/tag.cc Outdated
@junrushao junrushao self-assigned this Feb 5, 2021
@zxybazh zxybazh requested a review from junrushao February 5, 2021 18:26

@junrushao junrushao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thanks Xiyou for the first contribution 🎉

@junrushao

Copy link
Copy Markdown
Member

@comaniac Would you mind taking a second look?

@comaniac comaniac merged commit 4df530d into apache:main Feb 5, 2021
@comaniac

comaniac commented Feb 5, 2021

Copy link
Copy Markdown
Contributor

Thanks @zxybazh @junrushao1994
It seems to me that we should be consistent to let all fields have a default value (or not), but we could do this later.

@junrushao

Copy link
Copy Markdown
Member

@junrushao junrushao changed the title Add cuda tags and unit test [Target] Add cuda tags and unit test Feb 5, 2021
alexwong pushed a commit to alexwong/tvm that referenced this pull request Feb 11, 2021
* Add cuda tags and unit test

* Add missing space

* Remove extra indent

* Modify macro def position

* Fix clang format

* Fix clang format for set_config
Lokiiiiii pushed a commit to Lokiiiiii/tvm that referenced this pull request Mar 2, 2021
* Add cuda tags and unit test

* Add missing space

* Remove extra indent

* Modify macro def position

* Fix clang format

* Fix clang format for set_config
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Mar 2, 2021
* Add cuda tags and unit test

* Add missing space

* Remove extra indent

* Modify macro def position

* Fix clang format

* Fix clang format for set_config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants