[BUILD] Build system and dynamic library fixes #283
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A batch of fixes: - Added 'install' rule to cmake and make, which installs runtime headers and library (libtvm_runtime). - Added 'installdev' rule to make, which also installs the compiler infrastructure headers and library (libtvm) - Added 'INSTALL_DEV' option to cmake, for toggling installation of compiler infrastructure headers and library - cmake no longer builds into lib/ directory; instead all build products go in your build directory - New algorithm for dynamic library loading, as described in #281. Signed-off-by: Edward Z. Yang <ezyang@fb.com>
ezyang
force-pushed
the
pr/build-sys-loading
branch
from
9c49442 to
7be61ad
Compare
Member
|
Please fix the lint. Also, we will need to support .so suffix for macosx, or change makefile to generate .dylib files, either way is fine |
Contributor
Author
|
Is there a way to get access to the Jenkins? The link 404s for me. |
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
Merged
Member
|
We rely on github to do auth and currently there is a outrage, not sure if it is related |
tqchen
approved these changes
Jul 31, 2017
tqchen
changed the title
Member
|
cc @Javelinjs some followup changes of dylib loading in OSX might be needed, since now we build .dylib suffix libraries |
Member
|
I've made a modification and will make a pr soon |
tqchen
pushed a commit
to tqchen/tvm
that referenced
this pull request
May 26, 2018
tqchen
pushed a commit
that referenced
this pull request
May 29, 2018
tqchen
pushed a commit
to tqchen/tvm
that referenced
this pull request
Jul 6, 2018
sergei-mironov
pushed a commit
to sergei-mironov/tvm
that referenced
this pull request
Aug 8, 2018
vinx13
pushed a commit
to vinx13/tvm
that referenced
this pull request
Mar 9, 2022
junrushao
added a commit
to junrushao/tvm
that referenced
this pull request
Nov 25, 2025
Upstream : https://github.com/apache/tvm-ffi.git Branch : main New HEAD : d49effdb22392363050e1f2d85cd4b31bf242cf0 Subject : [Feature] Add cubin launcher utility as an extra header (apache#283) Author : Yaoyao Ding <dingyaoyao.cs@gmail.com> Date : 2025-11-25T18:35:33-05:00 Delta : 1 commit(s) since 8dbd28112cdd Compare : apache/tvm-ffi@8dbd281...d49effd This commit updates the tvm-ffi submodule to the latest upstream HEAD.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds an install rule, and fixes #281.
Let me know if I should add some docs; I wasn't sure where the right place was.