Todd Lipcon has submitted this change and it was merged.

Change subject: Fix LTO support
......................................................................


Fix LTO support

This fixes the CMake setup to be able to do LTO builds using lld and clang. I
was able to successfully enable LTO with this. However, the performance of
clang-3.9 + ThinLTO appeared to still be worse than devtoolset's gcc-4.9.2.

Nevertheless, it seemed like a good thing to commit to replace the currently
bogus flags.

Change-Id: I58a532dec43bd9ed5d7ea6419bc9dfc011f279b6
Reviewed-on: http://gerrit.cloudera.org:8080/6162
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <a...@cloudera.com>
---
M CMakeLists.txt
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I58a532dec43bd9ed5d7ea6419bc9dfc011f279b6
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to