From 8ffb26051f16a2ec4fbd1f8b0737d6d86c0ad819 Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Thu, 20 Oct 2016 15:45:02 -0400 Subject: [PATCH] Turn off buffer security checking in Release builds. I'm seeing /GS's _security_check_cookie() show up as a signficant piece of time when profiling. That's mostly just annoying noise. We generally use our Release builds for performance testing and Debug for correctness, so it seems like a fair thing to disable in Release builds... it's a sort of ASAN thing, which we only do in Debug on other platforms. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3782 Change-Id: I9b3cf4c5cf943fc2549f5bf91a1f6f7e41733e2c Reviewed-on: https://skia-review.googlesource.com/3782 Commit-Queue: Mike Klein Reviewed-by: Ben Wagner --- gn/BUILD.gn | 1 + 1 file changed, 1 insertion(+) diff --git a/gn/BUILD.gn b/gn/BUILD.gn index 9b3288a..bd5ce7f 100644 --- a/gn/BUILD.gn +++ b/gn/BUILD.gn @@ -332,6 +332,7 @@ config("release") { cflags = [ "/O2", "/Zc:inline", + "/GS-", ] ldflags = [ "/OPT:ICF", -- 2.7.4