Skip to content

build: update to clang-11#15021

Merged
lizan merged 16 commits intoenvoyproxy:mainfrom
lizan:clang11
Feb 24, 2021
Merged

build: update to clang-11#15021
lizan merged 16 commits intoenvoyproxy:mainfrom
lizan:clang11

Conversation

@lizan
Copy link
Member

@lizan lizan commented Feb 10, 2021

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@moderation
Copy link
Contributor

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Feb 11, 2021
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #15021 was synchronize by lizan.

see: more, trace.

Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@moderation
Copy link
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Feb 11, 2021
@lizan
Copy link
Member Author

lizan commented Feb 11, 2021

@moderation LLVM need to be upgraded together with WAVM.

@lambdai
Copy link
Contributor

lambdai commented Feb 11, 2021

@lizan What do I need to be done if I want to use machine native tool chains?

Install the clang-11, bazel/setup_clang.sh new clang11 dir and export CLANG_FORMAT=llvm-11/clang-format for sure.

What else?

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Feb 18, 2021
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@lizan lizan marked this pull request as ready for review February 19, 2021 05:50
Signed-off-by: Lizan Zhou <lizan@tetrate.io>
@lizan
Copy link
Member Author

lizan commented Feb 19, 2021

@lambdai yes that's it.

@lizan
Copy link
Member Author

lizan commented Feb 19, 2021

@htuch this is ready to go

Copy link
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! @moderation can we do these bumps as distinct followup PRs?

#
# This file was mostly manually assembled (with some hacky Python scripts) from
# clang+llvm-10.0.0-x86_64-linux-gnu-ubuntu-18.04.tar.xz and corresponding
# clang+llvm-11.0.0-x86_64-linux-gnu-ubuntu-16.04.tar.xz and corresponding
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we going backward from 18.04 to 16.04 here?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Either ubuntu 16.04 and 20.04 has prebuild 11.0.0 https://releases.llvm.org/download.html
Prebuild 11.0.0-ubuntu16.04 is running fine on my ubuntu 20.04 except lldb-11.
I guess we don't need lldb-11 during the build.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah that's because they don't have build for 18.04 and 16.04 would just work fine in 18.04.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fair enough; prefer going forward than back all things equal but not if there are complications.

@moderation
Copy link
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Feb 24, 2021
@lizan
Copy link
Member Author

lizan commented Feb 24, 2021

The clang-tidy error can be ignored. merging this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants