Tim Armstrong has posted comments on this change. Change subject: Update LLVM to 3.7 ......................................................................
Patch Set 2: They don't get unswitched for the most part since the scale and precision constants aren't injected into the codegened code. So it seems like somehow the code generated for remaining switches and functions is different between LLVM versions. Rather than try to figure out why, I'm just going to fix the underlying problem then rerun the benchmarks. -- To view, visit http://gerrit.cloudera.org:8080/2486 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I17d7afd05ad3b472a0bfe035bfc3daada5597b2d Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Skye Wanderman-Milne <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: No
