-
Notifications
You must be signed in to change notification settings - Fork 3.4k
images: Update LLVM to 18.1.8 #36197
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
Commit a344fe5 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 |
Relates: cilium/image-tools#302 Signed-off-by: Tam Mach <tam.mach@cilium.io>
a344fe5
to
25d5abe
Compare
Commit a344fe5 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 |
Signed-off-by: Cilium Imagebot <noreply@cilium.io>
/test |
This is to avoid the below issue while running integration tests. ``` clang: error while loading shared libraries: libtinfo.so.5: cannot open shared object file: No such file or directory ``` Signed-off-by: Tam Mach <tam.mach@cilium.io>
/test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Relay change LGTM
@sayboras can we close #32801 then, or are there still any outstanding tasks for the update to LVVM 18? I remember @gentoo-root did an evaluation of BPF program size / complexity changes when we last bumped to LLVM 17, to identify gains / regressions. |
yes, it will be the next step 👍 |
Relates: cilium/image-tools#302
Relates: #32801