Hello Abhishek Rawat, Joe McDonnell,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/20013

to look at the new patch set (#8).

Change subject: IMPALA-11459: Add LLVM 12
......................................................................

IMPALA-11459: Add LLVM 12

Add build for testing LLVM 12. LLVM 12 is a well-tested stable release
that predates function removal for opaque pointers.

Excludes building some parts of LLVM, similar to what Kudu does in
https://github.com/apache/kudu/blob/master/thirdparty/build-definitions.sh.

Carries run-clang-tidy patch to quiet script output. Other patches are
no longer necessary thanks to:
- https://github.com/llvm/llvm-project/commit/bebcd6fa4d2cb
- https://github.com/llvm/llvm-project/commit/965c5812f4380
- https://github.com/llvm/llvm-project/commit/3cbf3c8412890
- https://github.com/llvm/llvm-project/commit/383fe5c8668f6
- https://github.com/llvm/llvm-project/commit/18805ea951be0
- https://github.com/llvm/llvm-project/commit/947f969244083
- https://github.com/llvm/llvm-project/commit/9e68b734d6d0a
- https://github.com/llvm/llvm-project/commit/884040db08693
- https://github.com/llvm/llvm-project/commit/cc08a27d2ecc1

Makes cloudflarezlib/build.sh consistent with others to fix
  line 38: return: can only `return' from a function or sourced script

Excludes libicu from Boost for build environments where icu is
installed.

Testing: successful build on all target platforms (RHEL 7/8, Ubuntu
16.04/18.04/20.04, SLES 12/15).

Change-Id: Ida6911a5915707c32aab6b4424b6c6127ba984c0
---
M buildall.sh
M source/boost/build.sh
M source/cloudflarezlib/build.sh
M source/llvm/build-source-tarball.sh
A source/llvm/build-unified-tarball.sh
M source/llvm/build.sh
A 
source/llvm/llvm-12.0.1-patches/0001-Quiet-run-clang-tidy-invocation-output.patch
7 files changed, 300 insertions(+), 26 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/native-toolchain 
refs/changes/13/20013/8
--
To view, visit http://gerrit.cloudera.org:8080/20013
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ida6911a5915707c32aab6b4424b6c6127ba984c0
Gerrit-Change-Number: 20013
Gerrit-PatchSet: 8
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Abhishek Rawat <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>

Reply via email to