Don't omit frame pointers in profiling GN builds.

build/config/compiler/BUILD.gn

 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/config/android/config.gni")
 import("//build/config/chrome_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/nacl/config.gni")
 import("//build/toolchain/cc_wrapper.gni")
 import("//build/toolchain/toolchain.gni")
@@ skipping 223 matching lines @@
     # This is here so that all files get recompiled after a clang roll and
     # when turning clang on or off. (defines are passed via the command line,
     # and build system rebuild things when their commandline changes). Nothing
     # should ever read this define.
     defines += [ "CR_CLANG_REVISION=$clang_revision" ]
   }
 
   # Non-Mac Posix compiler flags setup.
   # -----------------------------------
   if (is_posix && !(is_mac || is_ios)) {
-    if (enable_profiling && !is_debug) {
-      cflags += [
-        "-fno-omit-frame-pointer",
-        "-g",
-      ]
+    if (enable_profiling) {
+      # Explicitly ask for frame pointers. Otherwise they are omitted when
+      # any optimization level is used (and Android debug builds use -Os).
+      cflags += [ "-fno-omit-frame-pointer" ]
+      if (!is_debug) {
+        cflags += [ "-g" ]
 
-      if (enable_full_stack_frames_for_profiling) {
-        cflags += [
-          "-fno-inline",
-          "-fno-optimize-sibling-calls",
-        ]
+        if (enable_full_stack_frames_for_profiling) {
+          cflags += [
+            "-fno-inline",
+            "-fno-optimize-sibling-calls",
+          ]
+        }
       }
     }
 
     defines += [ "_FILE_OFFSET_BITS=64" ]
 
     if (!is_android) {
       defines += [
         "_LARGEFILE_SOURCE",
         "_LARGEFILE64_SOURCE",
       ]
@@ skipping 937 matching lines @@
     # can be removed at link time with --gc-sections.
     "-fdata-sections",
     "-ffunction-sections",
   ]
   common_optimize_on_ldflags = []
 
   if (is_android) {
     # We don't omit frame pointers on arm64 since they are required
     # to correctly unwind stackframes which contain system library
     # function frames (crbug.com/391706).
-    if (!using_sanitizer && target_cpu != "arm64") {
+    if (!using_sanitizer && !enable_profiling && target_cpu != "arm64") {
       common_optimize_on_cflags += [ "-fomit-frame-pointer" ]
     }
 
     # TODO(jdduke) Re-enable on mips after resolving linking
     # issues with libc++ (crbug.com/456380).
     if (current_cpu != "mipsel" && current_cpu != "mips64el") {
       common_optimize_on_ldflags += [
         # Warn in case of text relocations.
         "-Wl,--warn-shared-textrel",
       ]
@@ skipping 69 matching lines @@
     # much even when optimization is disabled to get the binary size down.
     cflags = [
       "-Os",
       "-fdata-sections",
       "-ffunction-sections",
     ]
 
     # We don't omit frame pointers on arm64 since they are required
     # to correctly unwind stackframes which contain system library
     # function frames (crbug.com/391706).
-    if (!using_sanitizer && target_cpu != "arm64") {
+    if (!using_sanitizer && !enable_profiling && target_cpu != "arm64") {
       cflags += [ "-fomit-frame-pointer" ]
     }
 
     # Don't use gc-sections since it can cause links to succeed when they
     # actually shouldn't. http://crbug.com/159847
     ldflags = common_optimize_on_ldflags - [ "-Wl,--gc-sections" ]
   } else {
     cflags = [ "-O0" ]
     ldflags = []
   }
@@ skipping 140 matching lines @@
 
 if (is_ios || is_mac) {
   # On Mac and iOS, this enables support for ARC (automatic ref-counting).
   # See http://clang.llvm.org/docs/AutomaticReferenceCounting.html.
   config("enable_arc") {
     common_flags = [ "-fobjc-arc" ]
     cflags_objc = common_flags
     cflags_objcc = common_flags
   }
 }