-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[llvm] Pass FFI CMake options through to runtimes (for offload) #118807
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
Conversation
Pass the FFI-related CMake options through to runtimes, since offload is building against libffi. This is needed when the system requires custom `LIBFFI_INCLUDE` to build (e.g. on Gentoo where the headers are installed to `/usr/lib*/libffi/include`).
I hate the fact that we use FFI at all. I once had a patch to try to get rid of it #91264, but it ran into issues with the way Johannes routed 'implicit' arguments into the OpenMP kernels. I don't remember what they're used for, but they're put at the beginning of the buffer so it's impossible to optimize them out / ignore them. |
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/38/builds/1135 Here is the relevant piece of the build log for the reference
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/146/builds/1771 Here is the relevant piece of the build log for the reference
|
Pass the FFI-related CMake options through to runtimes, since offload is building against libffi. This is needed when the system requires custom
LIBFFI_INCLUDE
to build (e.g. on Gentoo where the headers are installed to/usr/lib*/libffi/include
).(this one's for non-standalone builds)