[SE] Stop using llvm-config --cxxflags
authorJason Henline <jhen@google.com>
Tue, 13 Sep 2016 15:44:18 +0000 (15:44 +0000)
committerJason Henline <jhen@google.com>
Tue, 13 Sep 2016 15:44:18 +0000 (15:44 +0000)
commit45b467523b34f0ad0fa03bd17a15a294d73ae808
treea737a6b962c7181929d78e0b782f027c15d8c45f
parent6339f1a028c7e9cc5150fcdd6b5363ee952e931f
[SE] Stop using llvm-config --cxxflags

Summary:
Build configuration was adding $(llvm-config --cxxflags) to the
StreamExecutor CXXFLAGS, but this was causing "-O3" to be passed even
for debug builds, and was making debugging difficult.

The llvm-config call was originally introduced to handle the -fno-rtti
flag because an RTTI StreamExecutor could not link with a no-RTTI LLVM.
This patch converts to using LLVM_ENABLE_RTTI and only adding the
`-fno-rtti` flag if needed, not all the rest of the LLVM CXXFLAGS.

I have tested this with clang-4.0 and gcc-4.8 on Ubuntu. Some work will
probably have to be done to support MSVC.

Reviewers: jlebar

Subscribers: beanz, jprice, parallel_libs-commits, mgorny

Differential Revision: https://reviews.llvm.org/D24474

llvm-svn: 281347
parallel-libs/streamexecutor/CMakeLists.txt