-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Return empty CSourceModule when no lowered_funcs exists in Relay mod #4847
Merged
Merged
Changes from 3 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
d6301d6
Use dummy func when no lowered_funcs exists in Relay mod
kumasento adbe5a2
Dummy func -> CSourceModule with empty code str
kumasento ca6e419
Added comments describing the empty CSouceModule
kumasento 2b955fc
Merge branch 'master' into dev-relay-dummy
kumasento cbaa76c
Always import external modules w/o assertions
kumasento 3e83a11
Use CSourceModule as a fallback for LLVMModule
kumasento a4f0d96
Changed cond for target == llvm
kumasento f1e8fed
Merge branch 'master' of https://github.com/dmlc/tvm into dev-relay-d…
kumasento d6656e2
Merge branch 'master' of https://github.com/dmlc/tvm into dev-relay-d…
kumasento 9441c52
Create an empty LLVM module w/o using dummy func
kumasento ccded94
Avoid using IR str concat to create LLVM module
kumasento bb55935
Improved comments for codegen.LLVMModuleCreate
kumasento 41450a5
Satisfy the linter for LLVMModuleCreate
kumasento File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
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.
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.
Sorry for the back and forth. Could you please add a comment here so that ppl would know what we are doing here?
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.
You can force push so that your previous CI could be terminated earlier.
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.
Sure thing, just added that