Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14817 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/14817 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- 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(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I50e09bc6f55c193e070e20b5b8ebb8e896fe37b3 Gerrit-Change-Number: 14817 Gerrit-PatchSet: 4 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <[email protected]>
