diff options
author | Jason Henline <jhen@google.com> | 2016-09-13 15:44:18 +0000 |
---|---|---|
committer | Jason Henline <jhen@google.com> | 2016-09-13 15:44:18 +0000 |
commit | 45b467523b34f0ad0fa03bd17a15a294d73ae808 (patch) | |
tree | a737a6b962c7181929d78e0b782f027c15d8c45f /llvm/lib/CodeGen/SafeStackLayout.cpp | |
parent | 6339f1a028c7e9cc5150fcdd6b5363ee952e931f (diff) | |
download | llvm-45b467523b34f0ad0fa03bd17a15a294d73ae808.zip llvm-45b467523b34f0ad0fa03bd17a15a294d73ae808.tar.gz llvm-45b467523b34f0ad0fa03bd17a15a294d73ae808.tar.bz2 |
[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
Diffstat (limited to 'llvm/lib/CodeGen/SafeStackLayout.cpp')
0 files changed, 0 insertions, 0 deletions