Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1147)

Unified Diff: build/config/compiler/BUILD.gn

Issue 2707223002: Make --enable-heap-profiling=native "work" on Windows. (Closed)
Patch Set: Add some TODOs Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« base/debug/stack_trace_unittest.cc ('K') | « base/debug/stack_trace_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/compiler/BUILD.gn
diff --git a/build/config/compiler/BUILD.gn b/build/config/compiler/BUILD.gn
index 1f8968670781cf0c6a810bffa6bb47170c0d8979..de110164da14306cf88d3715eb2f7ea500c66a46 100644
--- a/build/config/compiler/BUILD.gn
+++ b/build/config/compiler/BUILD.gn
@@ -1396,6 +1396,10 @@ config("default_stack_frames") {
} else if (is_android) {
cflags = [ "-fomit-frame-pointer" ]
}
+ } else if (is_win && enable_profiling) {
+ # common_optimize_on_cflags actually includes /Oy- to enable generation
+ # of frame pointers, but add it here explicitly in case that changes.
+ cflags = [ "/Oy-" ]
}
}
« base/debug/stack_trace_unittest.cc ('K') | « base/debug/stack_trace_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698