Re: [PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-25 Thread via GitHub
Lunderberg merged PR #15970: URL: https://github.com/apache/tvm/pull/15970 -- 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. To unsubscribe, e-mail:

Re: [PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-24 Thread via GitHub
Lunderberg commented on PR #15970: URL: https://github.com/apache/tvm/pull/15970#issuecomment-106256 > I'd love to get this PR in quickly as well as a hotfix, but seems i'm unable to get around the CI... I think there's a skip CI flag, though whether it is `[skipci]`,

Re: [PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-24 Thread via GitHub
junrushao commented on PR #15970: URL: https://github.com/apache/tvm/pull/15970#issuecomment-1777422369 I'd love to get this PR in quickly as well as a hotfix, but seems i'm unable to get around the CI... -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-24 Thread via GitHub
Lunderberg commented on PR #15970: URL: https://github.com/apache/tvm/pull/15970#issuecomment-1777207584 Whoops, looks like renaming the branch on github accidentally closed the PR. Thank you on the MSVC weirdnesses, and I've updated the PR to match the way all other C API functions

Re: [PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-24 Thread via GitHub
Lunderberg closed pull request #15970: [Hotfix] Mark python-FFI handling with TVM_DLL URL: https://github.com/apache/tvm/pull/15970 -- 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

Re: [PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-23 Thread via GitHub
junrushao commented on PR #15970: URL: https://github.com/apache/tvm/pull/15970#issuecomment-1776507511 We will further need to expose those methods in a `.h` file, wrapped with extern "C", and mark with TVM_DLL :-( MSVC is weird -- This is an automated message from the Apache Git

Re: [PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-23 Thread via GitHub
tqchen commented on PR #15970: URL: https://github.com/apache/tvm/pull/15970#issuecomment-1776406719 you want to place declaration in the header like the others and put TVM_DLL there -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[PR] [Hotfix] Mark python-FFI handling with TVM_DLL [tvm]

2023-10-23 Thread via GitHub
Lunderberg opened a new pull request, #15970: URL: https://github.com/apache/tvm/pull/15970 Bugfix for builds on Windows, which has missing symbol exports. -- 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