Skip to content

Fix TVM compile without LLVM - #7621

Merged
masahi merged 2 commits into
apache:mainfrom
trevor-m:no-llvm
Mar 10, 2021
Merged

Fix TVM compile without LLVM#7621
masahi merged 2 commits into
apache:mainfrom
trevor-m:no-llvm

Conversation

@trevor-m

@trevor-mtrevor-m commented Mar 9, 2021

Copy link
Copy Markdown
Contributor

#7398 introduced an compilation error when using USE_LLVM=OFF. Fix by adding preprocessor if guard.

tvm/src/target/metadata_module.cc:120:23: error: ‘CreateLLVMCrtMetadataModule’ was not declared in this scope target_module = CreateLLVMCrtMetadataModule(crt_exportable_modules, target);

@masahi
masahi merged commit f3d9cc1 into apache:mainMar 10, 2021
@tqchentqchen mentioned this pull request Mar 10, 2021
Closed
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request May 6, 2021
Trevor Morris
Fix TVM compile without LLVM (apache#7621)
94eadd5
* Fix TVM compile without LLVM
* Fix formatting
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request May 11, 2021
Trevor Morris@trevor-m
Fix TVM compile without LLVM (apache#7621)
8b89ca7
* Fix TVM compile without LLVM
* Fix formatting
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tqchentqchentqchen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@trevor-m@tqchen@masahi