-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set HIDE_PRIVATE_SYMBOLS ON for manylinux/osx/conda packages. #81
Conversation
@leandron we might also need that for conda packages and mac https://github.com/tlc-pack/tlcpack/blob/main/conda/recipe/build.sh |
* This is to workaround an issue caused by conflicting LLVM versions, first observed by since we updated Pytorch in TVM * Discussion at: apache/tvm#9362
3013b13
to
fb55487
Compare
I updated the PR to reflect that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @leandron, looks good to me!
This test was originally disabled due to the issue documented in apache#7455 affecting CI. I believe this has since been resolved by apache#9362. Note: This patch should not be merged until the changes in https: //github.com/tlc-pack/tlcpack/pull/81 are reflected in CI. Change-Id: Ib918595a1d9149e3c858ca761861304450cbfe13
This test was originally disabled due to the issue documented in apache#7455 affecting CI. I believe this has since been resolved by apache#9362. Note: This patch should not be merged until the changes in https: //github.com/tlc-pack/tlcpack/pull/81 are reflected in CI. Change-Id: Ib918595a1d9149e3c858ca761861304450cbfe13
This test was originally disabled due to the issue documented in #7455 affecting CI. I believe this has since been resolved by #9362. Note: This patch should not be merged until the changes in https: //github.com/tlc-pack/tlcpack/pull/81 are reflected in CI. Change-Id: Ib918595a1d9149e3c858ca761861304450cbfe13
This test was originally disabled due to the issue documented in apache#7455 affecting CI. I believe this has since been resolved by apache#9362. Note: This patch should not be merged until the changes in https: //github.com/tlc-pack/tlcpack/pull/81 are reflected in CI. Change-Id: Ib918595a1d9149e3c858ca761861304450cbfe13
This test was originally disabled due to the issue documented in apache#7455 affecting CI. I believe this has since been resolved by apache#9362. Note: This patch should not be merged until the changes in https: //github.com/tlc-pack/tlcpack/pull/81 are reflected in CI. Change-Id: Ib918595a1d9149e3c858ca761861304450cbfe13
This test was originally disabled due to the issue documented in apache#7455 affecting CI. I believe this has since been resolved by apache#9362. Note: This patch should not be merged until the changes in https: //github.com/tlc-pack/tlcpack/pull/81 are reflected in CI. Change-Id: Ib918595a1d9149e3c858ca761861304450cbfe13
This test was originally disabled due to the issue documented in apache#7455 affecting CI. I believe this has since been resolved by apache#9362. Note: This patch should not be merged until the changes in https: //github.com/tlc-pack/tlcpack/pull/81 are reflected in CI. Change-Id: Ib918595a1d9149e3c858ca761861304450cbfe13
This is to workaround an issue caused by conflicting LLVM
versions, first observed by since we updated Pytorch in TVM
Discussion at: [Bug] PyTorch and TVM loading problem due to conflicting LLVM symbols apache/tvm#9362