Skip to content

fix GPU other build#13235

Merged
driazati merged 2 commits intoapache:mainfrom
alter-xp:fix_gpu_other_build
Nov 1, 2022
Merged

fix GPU other build#13235
driazati merged 2 commits intoapache:mainfrom
alter-xp:fix_gpu_other_build

Conversation

@alter-xp
Copy link
Contributor

@alter-xp alter-xp commented Oct 31, 2022

@tvm-bot
Copy link
Collaborator

tvm-bot commented Oct 31, 2022

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

  • No users to auto-tag found, no teams are specified in PR title See #10317 for details
  • Built docs for commit 7138453 can be found here.

Generated by tvm-bot

@alter-xp alter-xp force-pushed the fix_gpu_other_build branch 2 times, most recently from 8d31e8a to 026ac9b Compare October 31, 2022 08:51
@alter-xp alter-xp force-pushed the fix_gpu_other_build branch from 026ac9b to 3764ac6 Compare November 1, 2022 01:24
Copy link
Member

@driazati driazati left a comment

Choose a reason for hiding this comment

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

thanks for the fix! one small comment on another cleanup we can do

Co-authored-by: driazati <9407960+driazati@users.noreply.github.com>
@alter-xp alter-xp force-pushed the fix_gpu_other_build branch from acf9cab to 7138453 Compare November 1, 2022 07:55
@driazati driazati merged commit e9e8c4b into apache:main Nov 1, 2022
@driazati
Copy link
Member

driazati commented Nov 1, 2022

fyi @alter-xp we had to revert this in #13261 since it was breaking CI, can you re-open and fix the issues? I'll have to sync your changes to a branch in apache/tvm to properly test them in CI before merging (the tvm-ci/branch job)

driazati added a commit that referenced this pull request Nov 1, 2022
alter-xp added a commit to alter-xp/tvm that referenced this pull request Nov 2, 2022
@alter-xp
Copy link
Contributor Author

alter-xp commented Nov 2, 2022

@driazati Oops, I'm sorry for the problem caused by this PR. let me fix it.

xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 10, 2022
fixes apache#12777

Co-authored-by: thead_iot_autotest <thead_iot_autotest@alibaba-inc.com>
Co-authored-by: driazati <9407960+driazati@users.noreply.github.com>
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 10, 2022
driazati pushed a commit that referenced this pull request Nov 16, 2022
Related pr #13235, #12777, #13261

because of ROCM build faild, pr #13235 destroyed CI.
this pr fixed that bug.
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
fixes apache#12777

Co-authored-by: thead_iot_autotest <thead_iot_autotest@alibaba-inc.com>
Co-authored-by: driazati <9407960+driazati@users.noreply.github.com>
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
Related pr apache#13235, apache#12777, apache#13261

because of ROCM build faild, pr apache#13235 destroyed CI.
this pr fixed that bug.
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.

[ci] gpu_other_build.sh isn't running

3 participants