Adar Dembo has posted comments on this change. Change subject: Bump LLVM to 4.0.0 ......................................................................
Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/7069/1//COMMIT_MSG Commit Message: Line 7: Bump LLVM to 4.0.0 Can you explain why you want to do this? I mean, it's generally a reasonable thing to do, but would be good to explain how this fixes the Thrift TSAN build (and link to the corresponding tsan patch, etc.) http://gerrit.cloudera.org:8080/#/c/7069/1/thirdparty/package-llvm.sh File thirdparty/package-llvm.sh: Line 1: #!/bin/bash Add copyright notice and a brief header explaining what this does. Line 5: VERSION=4.0.0 Make this settable via variable. Something like: VERSION=${VERSION:4.0.0} (apologies if that's not the right syntax). Maybe even omit the default since it's going to be wrong for the next upgrade anyway. Line 8: do Nit: move this to the previous line and separate with a semicolon; I think that's the bash style we tend to use elsewhere. http://gerrit.cloudera.org:8080/#/c/7069/1/thirdparty/vars.sh File thirdparty/vars.sh: PS1, Line 134: # Summary: : # 1. Unpack the llvm tarball : # 2. Unpack the clang tarball as tools/clang (rename from cfe-<version> to clang) : # 3. Unpack the extra clang tools tarball as tools/clang/tools/extra : # 4. Unpack the lld tarball as tools/lld : # 5. Unpack the compiler-rt tarball as projects/compiler-rt : # 6. Unpack the libc++ tarball as projects/libcxx : # 7. Unpack the libc++abi tarball as projects/libcxxabi : # 8. Create new tarball from the resulting source tree Move this description to package-llvm.sh. -- To view, visit http://gerrit.cloudera.org:8080/7069 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I64f98bb56d2510ba03ef5ea477a4d7bb4b48c555 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes