Backports for LLVM 7.1 [alt]#384
Conversation
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( I do have some suggestions for making it better though... For recipe/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/28249232299. Examine the logs at this URL for more detail. |
|
@conda-forge-admin, please rerender |
|
Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like there was nothing to do. This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/28219344078. Examine the logs at this URL for more detail. |
| - patches/llvm-lto-static.patch # [win] | ||
| - patches/partial-testing.patch | ||
| - patches/0001-pass-through-QEMU_LD_PREFIX-SDKROOT.patch | ||
| - patches/twine_cfg_undefined_behavior.patch | ||
| - patches/D47188-svml-VF.patch | ||
| # Revert an upstream change for Numba | ||
| # ref: https://github.com/numba/llvmlite/blob/v0.29.0/conda-recipes/0001-Revert-Limit-size-of-non-GlobalValue-name.patch | ||
| - patches/0001-Revert-Limit-size-of-non-GlobalValue-name.patch | ||
| # Fix getTypeName() for GCC 14+: move __PRETTY_FUNCTION__ outside namespace llvm | ||
| # so template args use fully-qualified names (e.g. llvm::Module not Module) | ||
| - patches/fix-typename-gcc14.patch |
There was a problem hiding this comment.
This is a mix of old patches from the 7.x branch, one current patch from main, and a new patch to make the tests pass with the current GCC.
| number: {{ build_number }} | ||
| skip: true # [win and vc<14] | ||
| number: 0 | ||
| skip: true # [osx or ppc64le] |
There was a problem hiding this comment.
We only care about Windows and Linux on ARM and x86; if someone wants a backport for other platforms they can ask.
| - python 3.7 | ||
| - cmake {{ cmake }} | ||
| - ninja | ||
| - python >=3,<3.13 |
There was a problem hiding this comment.
These old tests are not compatible with python 3.13 because pipe was removed from the standard library.
| - m2-coreutils # [win] | ||
| - m2-findutils # [win] | ||
| - m2-grep # [win] | ||
| - m2-sed # [win] |
There was a problem hiding this comment.
The tests require linux utils on Windows.
| -DLLVM_INSTALL_UTILS=ON ^ | ||
| -DLLVM_USE_SYMLINKS=OFF ^ | ||
| -DLLVM_UTILS_INSTALL_DIR=libexec\llvm ^ | ||
| -DLLVM_BUILD_LLVM_C_DYLIB=OFF ^ |
There was a problem hiding this comment.
LLVM_C is not available for Windows in this version.
…2026.06.25.08.09.32 Other tools: - conda-build 26.3.0 - rattler-build 0.64.1 - rattler-build-conda-compat 1.4.15
h-vetinari
left a comment
h-vetinari
left a comment
There was a problem hiding this comment.
There would be ways to clean this up further, but since this branch is so old, I don't consider that worth the time and effort on everyone's part. So LGTM. Thanks for trying to do this the "right" way, i.e. to avoid vendoring it. :)
I'll leave this open for a few days in case someone else from @conda-forge/llvmdev wants to opine.
|
@jakirkham, PTAL. |
3 participants
In this PR, I update the 7.x branch to LLVM 7.1 including backporting the new outputs from the main branch such as libllvm7.1, llvm-tools, llvmdev, lit, etc.
I modeled this the recipe on the main branch since that is what @h-vetinari requested.
The sonamed output is libllvm7.1 instead of libllvm7 because the shared library produced the build is libLLVM-7.1.so not libLLVM-7.so.
If this PR is rejected, we will be revending llvm 7.1 in numba-cuda-mlir.
Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)