Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
build: disable -fomit-frame-pointer on solaris
Browse files Browse the repository at this point in the history
This "optimization" cripples debuggability and has dubious performance value,
so we want to disable it at least on SmartOS.
  • Loading branch information
Dave Pacheco authored and bnoordhuis committed Feb 17, 2012
1 parent 977e211 commit 7864bb9
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion common.gypi
Expand Up @@ -34,8 +34,13 @@
},
},
'Release': {
'conditions': [
[ 'OS!="solaris"', {
'cflags': [ '-fomit-frame-pointer' ]
}],
],
# 'defines': [ 'NDEBUG' ],
'cflags': [ '-O3', '-fomit-frame-pointer', '-fdata-sections', '-ffunction-sections' ],
'cflags': [ '-O3', '-fdata-sections', '-ffunction-sections' ],
'conditions': [
['target_arch=="x64"', {
'msvs_configuration_platform': 'x64',
Expand Down

0 comments on commit 7864bb9

Please sign in to comment.