Casey Ching has posted comments on this change.

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


Patch Set 3: Code-Review+1

(2 comments)

http://gerrit.cloudera.org:8080/#/c/2546/3/source/llvm/build-source-tarball.sh
File source/llvm/build-source-tarball.sh:

Line 34:   tar xf ${THIS_DIR}/cfe-$PACKAGE_VERSION.src.tar.xz 
--use-compress-program xzcat
Looks like xzcat isnt available by default on a mac. Maybe do a special case 
for centos 5?


Line 45:   # Skip this on Centos 5.8 since it depends on perf_event.h.
Much better


-- 
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: 3
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