build: Enable no-omit-frame-pointer for all non-plain builds

This is useful for local installations with e.g. the `release`
buildtype.
Further more, Fedora has been doing the same system wide and it
apparently was helpful in various ways, see also

Also add -mno-omit-leaf-frame-pointer, like Fedora.

https://blogs.gnome.org/chergert/2023/10/03/what-have-frame-pointers-given-us-anyway/

Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/3333>
This commit is contained in:
Robert Mader 2023-10-18 08:25:21 +02:00 committed by Marge Bot
parent ee65ca791b
commit 5ffeb34251
4 changed files with 3 additions and 8 deletions

View File

@ -16,7 +16,6 @@ if buildtype != 'plain'
if get_option('debug')
clutter_debug_c_args += [
'-DCLUTTER_ENABLE_DEBUG',
'-fno-omit-frame-pointer',
]
else
clutter_debug_c_args += [

View File

@ -90,7 +90,6 @@ if buildtype != 'plain'
'-DCOGL_GL_DEBUG',
'-DCOGL_OBJECT_DEBUG',
'-DCOGL_ENABLE_DEBUG',
'-fno-omit-frame-pointer',
]
else
cogl_debug_c_args += [

View File

@ -467,6 +467,8 @@ add_project_arguments('-D_GNU_SOURCE', language: 'c')
buildtype = get_option('buildtype')
if buildtype != 'plain'
mutter_c_args = [
'-fno-omit-frame-pointer',
'-mno-omit-leaf-frame-pointer',
'-fno-strict-aliasing',
'-Wpointer-arith',
'-Wmissing-declarations',
@ -510,7 +512,6 @@ if buildtype != 'plain'
if get_option('debug')
mutter_c_args += [
'-DG_ENABLE_DEBUG',
'-fno-omit-frame-pointer',
]
endif

View File

@ -15,11 +15,7 @@ mtk_c_args = [
mtk_debug_c_args = []
if buildtype != 'plain'
if get_option('debug')
mtk_debug_c_args += [
'-fno-omit-frame-pointer'
]
else
if not get_option('debug')
mtk_debug_c_args += [
'-DG_DISABLE_ASSERT',
'-DG_DISABLE_CAST_CHECKS',