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

Unified Diff: chrome/installer/mini_installer/BUILD.gn

Issue 2018293002: Enable WPO for the x64 Official builds. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@4702
Patch Set: rebase Created 4 years, 7 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
« build/config/compiler/compiler.gni ('K') | « build/config/compiler/compiler.gni ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/mini_installer/BUILD.gn
diff --git a/chrome/installer/mini_installer/BUILD.gn b/chrome/installer/mini_installer/BUILD.gn
index a48f94ac7943648e5085d5f4164c1a87721e8853..aaa51672b0dfbd160c9fba33cfd0b141daa02ce8 100644
--- a/chrome/installer/mini_installer/BUILD.gn
+++ b/chrome/installer/mini_installer/BUILD.gn
@@ -167,6 +167,7 @@ executable("mini_installer") {
# This target is special so we manually override most linker flags and
# specify our own to keep the size down.
configs -= [
+ "//build/config/compiler:default_optimization",
grt (UTC plus 2) 2016/06/01 18:46:59 i don't get this. why?
Sébastien Marchand 2016/06/01 19:16:31 It's because the default optimization now adds the
brucedawson 2016/06/03 00:47:53 Well, as sometimes happens the reviewers successfu
Sébastien Marchand 2016/06/03 01:13:18 I think that this will add too much complexity to
"//build/config:executable_config",
"//build/config/win:console",
]
« build/config/compiler/compiler.gni ('K') | « build/config/compiler/compiler.gni ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698