-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
build: update LLVM to 14.0.0 #21135
build: update LLVM to 14.0.0 #21135
Conversation
Signed-off-by: Keith Smiley <keithbsmiley@gmail.com> Signed-off-by: Lizan Zhou <lizan@tetrate.io>
CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to |
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
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.
nice! thanks for doing all the real work here!
Risk Level: Low Testing: CI Docs Changes: Added Release Notes: Added Platform Specific Features: Signed-off-by: Keith Smiley <keithbsmiley@gmail.com> Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Risk Level: Low Testing: CI Docs Changes: Added Release Notes: Added Platform Specific Features: Signed-off-by: Keith Smiley <keithbsmiley@gmail.com> Signed-off-by: Lizan Zhou <lizan@tetrate.io> Signed-off-by: Ryan Northey <ryan@synca.io>
Recently CI saw errors like this: ``` bazel-out/k8-fastbuild/bin/external/org_llvm_releases_compiler_rt/_virtual_includes/fuzzed_data_provider/fuzzer/FuzzedDataProvider.h:194:38: error: no member named 'numeric_limits' in namespace 'std' return ConsumeIntegralInRange(std::numeric_limits<T>::min(), ~~~~~^ ``` Apparently this is due to a incompatibility between the `compiler-rt` dependency and the clang version we use (14.0). This is fixed by updating `compiler-rt` to match clang 14. Actually This commit cherry-picks the relevant changes done in upstream a while ago: envoyproxy/envoy#21135
Recently CI saw errors like this: ``` bazel-out/k8-fastbuild/bin/external/org_llvm_releases_compiler_rt/_virtual_includes/fuzzed_data_provider/fuzzer/FuzzedDataProvider.h:194:38: error: no member named 'numeric_limits' in namespace 'std' return ConsumeIntegralInRange(std::numeric_limits<T>::min(), ~~~~~^ ``` Apparently this is due to a incompatibility between the `compiler-rt` dependency and the clang version we use (14.0). This is fixed by updating `compiler-rt` to match clang 14. Actually This commit cherry-picks the relevant changes done in upstream a while ago: envoyproxy/envoy#21135
Signed-off-by: Keith Smiley keithbsmiley@gmail.com
Signed-off-by: Lizan Zhou lizan@tetrate.io
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]