Skip to content

Conversation

@ddwolf
Copy link
Contributor

@ddwolf ddwolf commented Jun 20, 2025

the exception is like this:

  File "/home/rss/duxiutao/repos/tvm/python/tvm/target/codegen.py", line 80, in llvm_lookup_intrinsic_id
    return _ffi_api.llvm_lookup_intrinsic_id(name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'tvm.target._ffi_api' has no attribute 'llvm_lookup_intrinsic_id'

@ddwolf ddwolf force-pushed the main branch 3 times, most recently from edf844d to 3cc152e Compare June 23, 2025 08:42
@cbalint13 cbalint13 self-assigned this Jun 23, 2025
Copy link
Contributor

@cbalint13 cbalint13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ddwolf ,

Thanks for the fix, this would enhance the user experience in the case of llvm absence.
Looks good, but wonder if we could not check the very llvm module presence instead.

@ddwolf ddwolf changed the title fix exception when tvm not built with llvm support [BugFix]fix exception when tvm not built with llvm support Jun 25, 2025
Copy link
Contributor

@cbalint13 cbalint13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ddwolf !

@cbalint13 cbalint13 merged commit a842004 into apache:main Jun 25, 2025
14 checks passed
ShiboXing pushed a commit to ShiboXing/tvm that referenced this pull request Aug 10, 2025
…8087)

Fix exception when tvm not built with llvm support.
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.

2 participants