keep frame pointer on macos

mjansson · Jul 26, 2021 · 3232ea3 · 3232ea3
1 parent c39da19
commit 3232ea3
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/build/ninja/clang.py b/build/ninja/clang.py
@@ -48,7 +48,7 @@ def initialize(self, project, archs, configs, includepaths, dependlibs, libpaths
     #Base flags
     self.cflags = ['-D' + project.upper() + '_COMPILE=1',
                    '-funit-at-a-time', '-fstrict-aliasing', '-fvisibility=hidden', '-fno-stack-protector',
-                   '-fomit-frame-pointer', '-fno-math-errno','-ffinite-math-only', '-funsafe-math-optimizations',
+                   '-fno-math-errno','-ffinite-math-only', '-funsafe-math-optimizations',
                    '-fno-trapping-math', '-ffast-math']
     self.cwarnflags = ['-W', '-Werror', '-pedantic', '-Wall', '-Weverything',
                        '-Wno-c++98-compat', '-Wno-padded', '-Wno-documentation-unknown-command',
@@ -84,8 +84,13 @@ def initialize(self, project, archs, configs, includepaths, dependlibs, libpaths
       self.cflags += ['-D_GNU_SOURCE=1']
       self.linkflags += ['-lpthread']
       self.oslibs += ['m']
-    if not self.target.is_windows():
+    if (not self.target.is_windows()) and (not self.target.is_macos()) and (not self.target.is_ios()):
       self.linkflags += ['-fomit-frame-pointer']
+    if self.target.is_macos() or self.target.is_ios():
+      # Needed for backtrace() system call (implemented as simple frame pointer walk by Apple)
+      self.cflags += ['-fno-omit-frame-pointer']
+    else:
+      self.cflags += ['-fomit-frame-pointer']
 
     self.includepaths = self.prefix_includepaths((includepaths or []) + ['.'])