[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-26 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 This is the same as the previous LLVM upgrade patch, e

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 1: Carry +2 from previous version of patch. -- To view, visit http://gerrit.cloudera.org:8080/2866 T

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2866 To unsubscribe, visit http://

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/2866 Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 This is the same as the p

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-25 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 1: This is the same as http://gerrit.cloudera.org/#/c/2486/ except without the dependency on libtinfo

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-23 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 This requires various changes for Impala to be fully f

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 15: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2486 To unsubscribe, visit http:/

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-22 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Internal Jenkins, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2486 to look at the new patch set (#15). Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 ...

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-22 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 14: Code-Review+2 Rebase, fix the LLVM versions in impala-config.sh and bootstrap_toolchain.py, and f

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-22 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2486 to look at the new patch set (#14). Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPAL

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 13: Code-Review+2 Carry +2 and +1. Will wait for toolchain changes to go in before merging. -- To vi

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-19 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2486 to look at the new patch set (#13). Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPAL

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-19 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 12: (4 comments) http://gerrit.cloudera.org:8080/#/c/2486/12/be/src/codegen/llvm-codegen-test.cc File

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-19 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 12: Code-Review+1 (2 comments) Some minor styling comments. Can you please address the question about th

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-19 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 12: (2 comments) Some minor comments. Will take one last pass in the morning. http://gerrit.cloudera.or

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2486 to look at the new patch set (#12). Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPALA-775,IMPALA

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 12: Code-Review+2 Carry dan's +2, will wait for +1 from kwho -- To view, visit http://gerrit.clouder

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 11: (4 comments) http://gerrit.cloudera.org:8080/#/c/2486/11/be/src/codegen/llvm-codegen.cc File be/s

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 11: Code-Review+2 (4 comments) Looks good to me. Please let Michael finish his review as well. http://ge

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 11: I also switched debug to use the release+asserts build. The debug llvm binaries are unreasonably

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#11). Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 This requires various changes for Impala to be fully functional wit

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 10: Addressed the comments, also fixed https://issues.cloudera.org/browse/IMPALA-3374 and a warning a

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. Patch Set 9: (15 comments) http://gerrit.cloudera.org:8080/#/c/2486/9/be/src/codegen/llvm-codegen.cc File be/sr

[Impala-CR](cdh5-trunk) IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0

2016-04-18 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#10). Change subject: IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 .. IMPALA-775,IMPALA-3374: Upgrade LLVM to 3.8.0 This requires various changes for Impala to be fully functional wit