diff options
| author | Kuba Mracek <mracek@apple.com> | 2017-03-27 17:14:48 +0000 |
|---|---|---|
| committer | Kuba Mracek <mracek@apple.com> | 2017-03-27 17:14:48 +0000 |
| commit | 90caf899eb86268e90efa8cf4b6fd7b770d9734b (patch) | |
| tree | 206010da6c5bf7444aec137aae903abf15ccadcf | |
| parent | afc9e3534371e07010752db09063669842763705 (diff) | |
| download | bcm5719-llvm-90caf899eb86268e90efa8cf4b6fd7b770d9734b.tar.gz bcm5719-llvm-90caf899eb86268e90efa8cf4b6fd7b770d9734b.zip | |
[sanitizers] Avoid using -fomit-frame-pointer on Darwin
On Darwin, we're having multiple issues with using -fomit-frame-pointer in the AddressSanitizer and ThreadSanitizer runtimes, so we're actually not using -fomit-frame-pointer in the our builds of the sanitizer dylibs. This patch just pushes our internal change upstream.
The issues are usually with debuggers, profilers and other tools that unwind the stack (crash reporter), which are often simply not able to get a stack trace. And crashlogs that don't contain a stack trace are a huge problem.
Differential Revision: https://reviews.llvm.org/D31376
llvm-svn: 298859
| -rw-r--r-- | compiler-rt/CMakeLists.txt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/compiler-rt/CMakeLists.txt b/compiler-rt/CMakeLists.txt index db083491f19..32358a1262c 100644 --- a/compiler-rt/CMakeLists.txt +++ b/compiler-rt/CMakeLists.txt @@ -124,7 +124,7 @@ if(NOT WIN32) endif() append_list_if(COMPILER_RT_HAS_FNO_BUILTIN_FLAG -fno-builtin SANITIZER_COMMON_CFLAGS) append_list_if(COMPILER_RT_HAS_FNO_EXCEPTIONS_FLAG -fno-exceptions SANITIZER_COMMON_CFLAGS) -if(NOT COMPILER_RT_DEBUG) +if(NOT COMPILER_RT_DEBUG AND NOT APPLE) append_list_if(COMPILER_RT_HAS_FOMIT_FRAME_POINTER_FLAG -fomit-frame-pointer SANITIZER_COMMON_CFLAGS) endif() append_list_if(COMPILER_RT_HAS_FUNWIND_TABLES_FLAG -funwind-tables SANITIZER_COMMON_CFLAGS) |
