Skip to content

[LIBXSMM] Add libxsmm to tvm ci - #10179

Merged
masahi merged 6 commits into
apache:mainfrom
zhuwenxi:add_libxsmm_to_tvm_ci
Mar 28, 2022
Merged

[LIBXSMM] Add libxsmm to tvm ci#10179
masahi merged 6 commits into
apache:mainfrom
zhuwenxi:add_libxsmm_to_tvm_ci

Conversation

@zhuwenxi

Copy link
Copy Markdown
Contributor

RFC: apache/tvm-rfcs#47

This is the first PR in the integration plan.

@zhuwenxi

Copy link
Copy Markdown
ContributorAuthor

@comaniac Could you help review this PR?

Comment threaddocker/install/ubuntu_install_libxsmm.sh Outdated
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
@masahi

Copy link
Copy Markdown
Member

@zhuwenxi please run another job. I can take care of the image update. Note that changing the docker script alone doesn't update the CI environment.

@masahi

Copy link
Copy Markdown
Member

ping @zhuwenxi please send another CI job, I want to update the cpu image together with onednn update in #10266

@zhuwenxi

Copy link
Copy Markdown
ContributorAuthor

@zhuwenxi please run another job. I can take care of the image update. Note that changing the docker script alone doesn't update the CI environment.

Thank you @masahi for the notice that changing docker script alone is not enough. I'm wondering is there any PR I can refer to fully update the CI?

@masahi
masahi merged commit 76c7677 into apache:mainMar 28, 2022
@masahi

Copy link
Copy Markdown
Member

The actual update needs to be done by a committer. I can do that this week.

@masahi

Copy link
Copy Markdown
Member

The CI image has been updated by #10889. We are ready to use libxsmm on CI.

pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
* [LIBXSMM] add libxsmm to TVM CI.
* Config "make" thread number in a more flexible way.
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
* Empty commit to trigger github CI.
* Update ubuntu_install_libxsmm.sh.
* Trigger CI tasks.
* Trigger CI tasks.
Co-authored-by: wenxizhu <wenxizhu@tencent.com>
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
Sign up for freeto 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

@zhuwenxi@masahi@comaniac