Skip to content
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

[Frontend] Check LLVM enabled/installed #8414

Merged
merged 1 commit into from
Jul 9, 2021
Merged

Conversation

kueitang
Copy link
Contributor

@kueitang kueitang commented Jul 7, 2021

  • Some ops(ex:view) call infer_value when converting a model into Relay IR.
  • If LLVM is not enabled, it leads to segmentation fault.
  • infer_value() (tvm/python/tvm/relay/frontend/common.py) uses tvm.relay.build and it assumes a LLVM target.

Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.
@tqchen

-Some ops(ex:view) call infer_value when converting a model into Relay IR.
-If LLVM is not enabled, it leads to segementation fault.
@kueitang kueitang closed this Jul 8, 2021
@kueitang kueitang reopened this Jul 8, 2021
@jcf94 jcf94 merged commit 53cb8aa into apache:main Jul 9, 2021
@kueitang kueitang deleted the debug-info branch July 27, 2021 06:59
ylc pushed a commit to ylc/tvm that referenced this pull request Sep 29, 2021
-Some ops(ex:view) call infer_value when converting a model into Relay IR.
-If LLVM is not enabled, it leads to segementation fault.

Co-authored-by: kueitang <kueitang@qti.qualcomm.com>
zxy844288792 pushed a commit to zxy844288792/tvm that referenced this pull request Mar 4, 2022
-Some ops(ex:view) call infer_value when converting a model into Relay IR.
-If LLVM is not enabled, it leads to segementation fault.

Co-authored-by: kueitang <kueitang@qti.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants