Dan Hecht has posted comments on this change.

Change subject: Support for building LLVM 3.7 and 3.8 on CentOS 5
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2546/4//COMMIT_MSG
Commit Message:

Line 14: and also provides an alternative to the gcc runtime C/C++ runtime 
libraries.
please clarify that we are only doing without it on CentOS 5.8 (as opposed to 
always).  Also, what exactly is the consequence? That on Centos5.8 we can't run 
with sanitizes? What's the consequence of using the gcc runtime?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iad649363fc64f30e0f17a9d51d7694a6fc00bc12
Gerrit-PatchSet: 4
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Casey Ching <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: Yes

Reply via email to