Skip to content

GH-39928: [C++][Gandiva] Accept LLVM 18#39934

Merged
kou merged 1 commit intoapache:mainfrom
kou:cpp-llvm-18
Feb 5, 2024
Merged

GH-39928: [C++][Gandiva] Accept LLVM 18#39934
kou merged 1 commit intoapache:mainfrom
kou:cpp-llvm-18

Conversation

@kou
Copy link
Copy Markdown
Member

@kou kou commented Feb 4, 2024

Rationale for this change

LLVM 18.1 will be released soon.

What changes are included in this PR?

Accept LLVM 18.1.

Are these changes tested?

Yes.

Are there any user-facing changes?

Yes.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 4, 2024

⚠️ GitHub issue #39928 has been automatically assigned in GitHub to PR creator.

@kou
Copy link
Copy Markdown
Member Author

kou commented Feb 4, 2024

@github-actions crossbow submit ubuntu-noble-amd64

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Feb 4, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 4, 2024

Revision: 8bb0ca0

Submitted crossbow builds: ursacomputing/crossbow @ actions-7c9965348b

Task Status
ubuntu-noble-amd64 GitHub Actions

@kou
Copy link
Copy Markdown
Member Author

kou commented Feb 4, 2024

@niyue Do you want to review this?

@niyue
Copy link
Copy Markdown
Contributor

niyue commented Feb 5, 2024

LGTM

@kou kou merged commit ed78986 into apache:main Feb 5, 2024
@kou kou deleted the cpp-llvm-18 branch February 5, 2024 02:51
@kou kou removed the awaiting committer review Awaiting committer review label Feb 5, 2024
@conbench-apache-arrow
Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit ed78986.

There was 1 benchmark result indicating a performance regression:

The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them.

dgreiss pushed a commit to dgreiss/arrow that referenced this pull request Feb 19, 2024
### Rationale for this change

LLVM 18.1 will be released soon.

### What changes are included in this PR?

Accept LLVM 18.1.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: apache#39928

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[C++][Gandiva] Support LLVM-18

2 participants