Hello Kudu Jenkins, Adar Dembo,

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

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

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

Change subject: Update to LLVM 9.0.0
......................................................................

Update to LLVM 9.0.0

This bumps to the latest LLVM release, just for the sake of being up to
date with the latest warnings and code generation improvements. I
checked codegen-test for any obvious regressions in instruction counts
or inlining behavior.

This added some new UBSAN checks. I disabled implicit integer conversion
and truncation, which seems to be an issue in a lot of spots in our code
(particularly StringPiece from gutil), but did fix one issue where it
spotted an actual lossy type truncation from uint64 to uint32.

Change-Id: I50e09bc6f55c193e070e20b5b8ebb8e896fe37b3
---
M CMakeLists.txt
M src/kudu/rpc/rpc_introspection.proto
M thirdparty/build-definitions.sh
M thirdparty/download-thirdparty.sh
D thirdparty/patches/llvm-fix-944-destruction-of-a-locked-mutex.patch
D thirdparty/patches/llvm-fix-amazon-linux.patch
M thirdparty/patches/llvm-iwyu-include-picker.patch
D thirdparty/patches/llvm-iwyu-llvm-6-compat.patch
D thirdparty/patches/llvm-iwyu-nocurses.patch
D 
thirdparty/patches/llvm-tsan-disable-trace-switching-after-multithreaded-for.patch
D thirdparty/patches/llvm-ustat-removal.patch
M thirdparty/vars.sh
12 files changed, 8 insertions(+), 342 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/17/14817/3
--
To view, visit http://gerrit.cloudera.org:8080/14817
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I50e09bc6f55c193e070e20b5b8ebb8e896fe37b3
Gerrit-Change-Number: 14817
Gerrit-PatchSet: 3
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to