diff options
author | 2016-10-19 22:24:10 -0400 | |
---|---|---|
committer | 2016-10-20 17:01:14 +0000 | |
commit | 0bcfeac25104127192eaa2236ec03b5b936af160 (patch) | |
tree | a5c2b60929e887ba014d4ab091379a86e686d81a /gn | |
parent | 2878e76247fd335dee755cfd4c9cd026804f9655 (diff) |
Compile Release builds with -momit-leaf-frame-pointer.
Unlike -fomit-frame-pointer, this doesn't make debugging or profiling any more difficult, as it only applies to leaves. It will make our code (negligibly) smaller and (negligibly) faster.
Mostly I just find it easier to read the disassembly without all the rbp gymnastics getting in the way.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3700
Change-Id: I4b96aee7619791d5980de7f46e82836ca08a6456
Reviewed-on: https://skia-review.googlesource.com/3700
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'gn')
-rw-r--r-- | gn/BUILD.gn | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/gn/BUILD.gn b/gn/BUILD.gn index 5d6a82ae38..24807d89b5 100644 --- a/gn/BUILD.gn +++ b/gn/BUILD.gn @@ -331,7 +331,10 @@ config("release") { if (is_win) { cflags = [ "/O2" ] } else { - cflags = [ "-O3" ] + cflags = [ + "-O3", + "-momit-leaf-frame-pointer", + ] } defines = [ "NDEBUG" ] } |