Skip to content

images: Update LLVM to 18.1.6 #32816

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

images: Update LLVM to 18.1.6 #32816

wants to merge 3 commits into from

Conversation

gentoo-root
Copy link
Contributor

Fixes: #32801

Update LLVM to 18.1.6

@gentoo-root gentoo-root added the release-note/misc This PR makes changes that have no direct user impact. label May 31, 2024
@gentoo-root gentoo-root requested review from a team as code owners May 31, 2024 11:15
@gentoo-root gentoo-root requested review from borkmann, aanm and nebril May 31, 2024 11:15
@gentoo-root gentoo-root temporarily deployed to release-base-images May 31, 2024 11:15 — with GitHub Actions Inactive
@auto-committer auto-committer bot requested review from a team as code owners May 31, 2024 11:29
@auto-committer auto-committer bot requested a review from kaworu May 31, 2024 11:29
@auto-committer auto-committer bot temporarily deployed to release-base-images May 31, 2024 11:30 Inactive
@kaworu kaworu added the dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. label May 31, 2024
@gentoo-root gentoo-root force-pushed the pr/max/upgrade-llvm-18-1-6 branch from f3b97e2 to 68d45a2 Compare May 31, 2024 15:32
@gentoo-root gentoo-root temporarily deployed to release-base-images May 31, 2024 15:32 — with GitHub Actions Inactive
@gentoo-root gentoo-root removed the dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. label May 31, 2024
@github-merge-queue github-merge-queue bot temporarily deployed to release-base-images May 31, 2024 16:08 Inactive
@gentoo-root
Copy link
Contributor Author

/test

@gentoo-root gentoo-root force-pushed the pr/max/upgrade-llvm-18-1-6 branch from 947ef53 to 91fa8c7 Compare June 3, 2024 19:15
@gentoo-root gentoo-root requested a review from a team as a code owner June 3, 2024 19:15
@maintainer-s-little-helper
Copy link

Commit c7c0e3d does not match "(?m)^Signed-off-by:".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. label Jun 3, 2024
@gentoo-root gentoo-root temporarily deployed to release-base-images June 3, 2024 19:15 — with GitHub Actions Inactive
@gentoo-root
Copy link
Contributor Author

/ci-verifier

@gentoo-root gentoo-root force-pushed the pr/max/upgrade-llvm-18-1-6 branch from 91fa8c7 to acb0b14 Compare June 4, 2024 16:22
@maintainer-s-little-helper
Copy link

Commit c7c0e3d does not match "(?m)^Signed-off-by:".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

This was referenced Aug 27, 2024
Closed
Merged
@github-actions GitHub Actions
Copy link

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Sep 27, 2024
@kaworu kaworu added the dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. label Sep 27, 2024
@gentoo-root gentoo-root removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Sep 27, 2024
@github-actions GitHub Actions
Copy link

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Oct 28, 2024
@gentoo-root gentoo-root removed the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Oct 28, 2024
@gentoo-root gentoo-root force-pushed the pr/max/upgrade-llvm-18-1-6 branch from 95c596b to ae35928 Compare October 28, 2024 09:14
@gentoo-root gentoo-root temporarily deployed to release-base-images October 28, 2024 09:14 — with GitHub Actions Inactive
@auto-committer auto-committer bot temporarily deployed to release-base-images October 28, 2024 09:40 Inactive
@auto-committer auto-committer bot temporarily deployed to release-base-images October 28, 2024 09:45 Inactive
@gentoo-root
Copy link
Contributor Author

/test

@gentoo-root gentoo-root force-pushed the pr/max/upgrade-llvm-18-1-6 branch from c2b5e6e to de5dbf8 Compare October 28, 2024 10:58
@gentoo-root gentoo-root temporarily deployed to release-base-images October 28, 2024 10:58 — with GitHub Actions Inactive
@gentoo-root gentoo-root temporarily deployed to release-base-images October 28, 2024 10:59 — with GitHub Actions Inactive
gentoo-root and others added 3 commits October 28, 2024 13:08
@gentoo-root
bpf: Add missing __align_stack_8 in encap_geneve_dsr_opt4
5148255 
LLVM 18 doesn't align these structs to 8 by default, and our memcpy
implementation fails to pass the verifier when applied to these structs,
because the verifier requires stack access to be aligned. Align all
affected structs.

Ref: 4853fb1 ("bpf: Use __align_stack_8 on IPv6 structs")
Fixes: c6a2836 ("bpf: dsr: refactor encap_geneve_dsr_opt()")
Signed-off-by: Maxim Mikityanskiy <maxim@isovalent.com>
@gentoo-root
images: Update LLVM to 18.1.6
b1d425c 
Use LLVM 18.1.4 where install-llvm-action is involved, because LLVM
upstream doesn't provide binaries of 18.1.6 for x86_64 Linux at the
moment.

Fixes: #32801
Signed-off-by: Maxim Mikityanskiy <maxim@isovalent.com>
@gentoo-root
images: update cilium-{runtime,builder}
cfea09a 
Signed-off-by: Cilium Imagebot <noreply@cilium.io>
@gentoo-root gentoo-root force-pushed the pr/max/upgrade-llvm-18-1-6 branch from de5dbf8 to cfea09a Compare October 28, 2024 11:08
@gentoo-root gentoo-root temporarily deployed to release-base-images October 28, 2024 11:08 — with GitHub Actions Inactive
@gentoo-root gentoo-root temporarily deployed to release-base-images October 28, 2024 11:09 — with GitHub Actions Inactive
@gentoo-root
Copy link
Contributor Author

/test

@gentoo-root gentoo-root mentioned this pull request Oct 28, 2024
Merged
@github-actions GitHub Actions
Copy link

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label Nov 28, 2024
@github-actions GitHub Actions
Copy link

This pull request has not seen any activity since it was marked stale.
Closing.

@github-actions github-actions bot closed this Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaworu kaworu kaworu approved these changes

@borkmann borkmann borkmann approved these changes

@nebril nebril nebril approved these changes

@aanm aanm aanm approved these changes

Assignees
No one assigned
Labels
area/llvm Requires upstream work in LLVM. dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. dont-merge/preview-only Only for preview or testing, don't merge it. release-note/misc This PR makes changes that have no direct user impact. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Upgrade to LLVM 18.1.8
6 participants
@gentoo-root @aanm @julianwiedmann @kaworu @borkmann @nebril