Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15058 )

Change subject: build: restrict clang version, prefer lld, enable thinlto
......................................................................


Patch Set 4: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15058/4/cmake_modules/KuduLinker.cmake
File cmake_modules/KuduLinker.cmake:

http://gerrit.cloudera.org:8080/#/c/15058/4/cmake_modules/KuduLinker.cmake@68
PS4, Line 68:     # [1] https://reviews.llvm.org/D63974
Might prefer to link to the actual bug itself rather than the CR: 
https://bugs.llvm.org/show_bug.cgi?id=42442.



--
To view, visit http://gerrit.cloudera.org:8080/15058
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I09ded0c44c9f7a6839489f0abf5baa4eaf1971f0
Gerrit-Change-Number: 15058
Gerrit-PatchSet: 4
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, 21 Jan 2020 22:37:23 +0000
Gerrit-HasComments: Yes

Reply via email to