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 <mtklein@chromium.org>
Reviewed-by: Ben Wagner <bungeman@google.com>

This commit is contained in:

Mike Klein

2016-10-20 15:45:02 -04:00

parent 34a2ca1747

commit 8ffb26051f

1 changed files with 1 additions and 0 deletions

1

gn/BUILD.gn

View File

 @ -332,6 +332,7 @@ config("release") {
     cflags = [
       "/O2",
       "/Zc:inline",
       "/GS-",
     ]
     ldflags = [
       "/OPT:ICF",

Turn off buffer security checking in Release builds.

1 gn/BUILD.gn Unescape Escape View File

1

gn/BUILD.gn

View File