Todd Lipcon has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14817 )

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


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/14817/2/thirdparty/build-definitions.sh
File thirdparty/build-definitions.sh:

http://gerrit.cloudera.org:8080/#/c/14817/2/thirdparty/build-definitions.sh@103
PS2, Line 103:
> Don't need to update libcxxabi/libcxx builds at all?
apparently not? we'll see if the build passes on jenkins


http://gerrit.cloudera.org:8080/#/c/14817/2/thirdparty/build-definitions.sh@320
PS2, Line 320:     -DLLVM_TEMPORARILY_ALLOW_OLD_TOOLCHAIN=ON \
> So does this mean we'll need to bump our gcc requirement to support newer v
yea, it looks like the minimum recommended now is GCC 5.1 devtoolset-3 has 4.9, 
so it's too old. I don't see any reason we wouldn't go to the latest 
(devtoolset-8) which has gcc 8.3.

Older Ubuntu is probably the trickier thing to deal with - probably need to 
find some old PPA, or download and use a clang binary toolchain to bootstrap 
things.



--
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: comment
Gerrit-Change-Id: I50e09bc6f55c193e070e20b5b8ebb8e896fe37b3
Gerrit-Change-Number: 14817
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Comment-Date: Tue, 03 Dec 2019 19:28:00 +0000
Gerrit-HasComments: Yes

Reply via email to