Reland "always optimize third_party code"

This is a reland of c766370d86

Original change's description:
> always optimize third_party code
> 
> Change-Id: I5b2244460a4760e9336640f597d0f74c374a0d04
> Reviewed-on: https://skia-review.googlesource.com/155641
> Commit-Queue: Mike Klein <mtklein@google.com>
> Commit-Queue: Brian Osman <brianosman@google.com>
> Auto-Submit: Mike Klein <mtklein@google.com>
> Reviewed-by: Brian Osman <brianosman@google.com>

Change-Id: I63e7f9ca852fc99728d7a01d9987b3506115d266
Reviewed-on: https://skia-review.googlesource.com/155760
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Klein <mtklein@google.com>
This commit is contained in:
Mike Klein 2018-09-19 13:44:43 -04:00 committed by Skia Commit-Bot
parent 44c6167c41
commit 7ea977bc5e
3 changed files with 11 additions and 2 deletions

View File

@ -455,7 +455,7 @@ config("no_rtti") {
}
}
config("release") {
config("optimize") {
if (is_win) {
cflags = [
"/O2",
@ -484,6 +484,9 @@ config("release") {
ldflags += [ "-O3" ]
}
}
}
config("NDEBUG") {
defines = [ "NDEBUG" ]
}

View File

@ -218,7 +218,10 @@ default_configs = [
"//gn:no_rtti",
]
if (!is_debug) {
default_configs += [ "//gn:release" ]
default_configs += [
"//gn:optimize",
"//gn:NDEBUG",
]
}
if (!is_official_build) {
default_configs += [

View File

@ -46,6 +46,9 @@ set_defaults("third_party") {
# Official builds don't have warnings to begin with.
configs -= [ "//gn:warnings" ]
}
if (is_debug) {
configs += [ "//gn:optimize" ]
}
}
template("system") {