Skip to content

Conversation

@ZuseZ4
Copy link
Collaborator

@ZuseZ4 ZuseZ4 commented Feb 20, 2025

fixes #2247
I tested locally that it correctly recognizes the LLVM_ENABLE_PLUGINS variable.
In combination with the other patch it allows building me a local enzyme against an llvm without the plugin interface.

image

@ZuseZ4 ZuseZ4 requested a review from wsmoses February 20, 2025 03:30
Copy link
Member

@wsmoses wsmoses left a comment

Choose a reason for hiding this comment

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

lgtm modulo some name changes

@ZuseZ4
Copy link
Collaborator Author

ZuseZ4 commented Feb 21, 2025

image
It catches the case of -DENZYME_EXTERNAL_SHARED_LIB=OFF in combination with an LLVM without the plugin interface.

@wsmoses wsmoses merged commit 5004a8f into main Feb 21, 2025
10 of 23 checks passed
@wsmoses wsmoses deleted the no-plugin-support branch February 21, 2025 22:31
wsmoses added a commit that referenced this pull request Feb 26, 2025
wsmoses added a commit that referenced this pull request Feb 26, 2025
wsmoses added a commit that referenced this pull request Feb 26, 2025
wsmoses added a commit that referenced this pull request Feb 26, 2025
* Fix bugs introduced by CMake flag in #2249

* macos

* Update enzyme-ci.yml

* Try macos lit path
gbaraldi pushed a commit to gbaraldi/Enzyme that referenced this pull request Jun 27, 2025
* Support jl_genericmemory_copyto

* fix

* fix

* Update array.jl
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.

Build Enzyme without LLVM Plugin Interface?

3 participants