GH-47469: [C++][Gandiva] Add support for LLVM 21.1.0#47473
Conversation
|
|
|
@github-actions crossbow submit -g cpp |
|
Revision: c0e329e Submitted crossbow builds: ursacomputing/crossbow @ actions-e1bf7ac3c1 |
|
The CPP Valgrind failure is unrelated and seems to have been around for a while. I've opened an issue as I don't think it's tracked anywhere: |
|
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 57b4b4b. There weren't enough matching historic benchmark results to make a call on whether there were regressions. The full Conbench report has more details. |
|
Thank you for this, Raúl. |
) LLVM 21.1 was released and some of our CI jobs are failing. * Accept LLVM 21.1 * Don't use deprecated API Two of the related commits that changed APIs on LLVM are: - llvm/llvm-project@034f2b3 - llvm/llvm-project@b18e5b6 Yes on CI No * GitHub Issue: apache#47469 Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
) LLVM 21.1 was released and some of our CI jobs are failing. * Accept LLVM 21.1 * Don't use deprecated API Two of the related commits that changed APIs on LLVM are: - llvm/llvm-project@034f2b3 - llvm/llvm-project@b18e5b6 Yes on CI No * GitHub Issue: apache#47469 Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Rationale for this change
LLVM 21.1 was released and some of our CI jobs are failing.
What changes are included in this PR?
Two of the related commits that changed APIs on LLVM are:
Are these changes tested?
Yes on CI
Are there any user-facing changes?
No