Skip to content

Fixup libtorch backend build#13320

Merged
junrushao merged 1 commit intoapache:mainfrom
leiwen83:libtorch_fixup
Nov 8, 2022
Merged

Fixup libtorch backend build#13320
junrushao merged 1 commit intoapache:mainfrom
leiwen83:libtorch_fixup

Conversation

@leiwen83
Copy link
Copy Markdown
Contributor

@leiwen83 leiwen83 commented Nov 8, 2022

Add clang-format disable for header to prevent reorder. Torch header file need to be put at the end since torch's dlpack is a little different with tvm's.

cc @t-vi @masahi

Signed-off-by: Lei Wen wenlei03@qiyi.com

@tvm-bot
Copy link
Copy Markdown
Collaborator

tvm-bot commented Nov 8, 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 f4e6e7d can be found here.

Generated by tvm-bot

@github-actions github-actions bot requested a review from masahi November 8, 2022 03:24
Add clang-format disable for header to prevent reorder.
Torch header file need to be put at the end since torch's dlpack
is a little different with tvm's.

cc @t-vi @masahi

Signed-off-by: Lei Wen <wenlei03@qiyi.com>
Copy link
Copy Markdown
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown

@shingjan shingjan left a comment

Choose a reason for hiding this comment

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

LGTM. Could there be more context on the dlpack mismatch between PyTorch and TVM. Thanks!

@junrushao junrushao merged commit bf77e79 into apache:main Nov 8, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 10, 2022
Add clang-format disable for header to prevent reorder.
Torch header file need to be put at the end since torch's dlpack
is a little different with tvm's.

Signed-off-by: Lei Wen <wenlei03@qiyi.com>
Co-authored-by: Lei Wen <wenlei03@qiyi.com>
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
Add clang-format disable for header to prevent reorder.
Torch header file need to be put at the end since torch's dlpack
is a little different with tvm's.

Signed-off-by: Lei Wen <wenlei03@qiyi.com>
Co-authored-by: Lei Wen <wenlei03@qiyi.com>
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.

5 participants