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

Side by Side Diff: skia/BUILD.gn

Issue 2270693006: Adjust skia build settings to match gyp, improve perf (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Another place to add optimize_max to Windows as well Created 4 years, 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/features.gni") 5 import("//build/config/features.gni")
6 import("//build/config/ui.gni") 6 import("//build/config/ui.gni")
7 import("//testing/test.gni") 7 import("//testing/test.gni")
8 if (current_cpu == "arm") { 8 if (current_cpu == "arm") {
9 import("//build/config/arm.gni") 9 import("//build/config/arm.gni")
10 } 10 }
(...skipping 495 matching lines...) Expand 10 before | Expand all | Expand 10 after
506 ] 506 ]
507 } 507 }
508 508
509 if (skia_support_pdf) { 509 if (skia_support_pdf) {
510 deps += [ "//third_party/sfntly" ] 510 deps += [ "//third_party/sfntly" ]
511 sources += gypi_skia_pdf.sources 511 sources += gypi_skia_pdf.sources
512 } else { 512 } else {
513 sources += [ "//third_party/skia/src/pdf/SkDocument_PDF_None.cpp" ] 513 sources += [ "//third_party/skia/src/pdf/SkDocument_PDF_None.cpp" ]
514 } 514 }
515 515
516 if (is_android && !is_debug) { 516 if ((is_android || is_win) && !is_debug) {
517 configs -= [ "//build/config/compiler:default_optimization" ] 517 configs -= [ "//build/config/compiler:default_optimization" ]
518 configs += [ "//build/config/compiler:optimize_max" ] 518 configs += [ "//build/config/compiler:optimize_max" ]
519 } 519 }
520 520
521 if (is_mac) { 521 if (is_mac) {
522 libs = [ 522 libs = [
523 "AppKit.framework", 523 "AppKit.framework",
524 "CoreFoundation.framework", 524 "CoreFoundation.framework",
525 "CoreGraphics.framework", 525 "CoreGraphics.framework",
526 "CoreText.framework", 526 "CoreText.framework",
(...skipping 165 matching lines...) Expand 10 before | Expand all | Expand 10 after
692 } else { 692 } else {
693 sources = gypi_skia_opts.none_sources 693 sources = gypi_skia_opts.none_sources
694 } 694 }
695 } else if (current_cpu == "mips64el") { 695 } else if (current_cpu == "mips64el") {
696 cflags += [ "-fomit-frame-pointer" ] 696 cflags += [ "-fomit-frame-pointer" ]
697 sources = gypi_skia_opts.none_sources 697 sources = gypi_skia_opts.none_sources
698 } else { 698 } else {
699 assert(false, "Need to port cpu specific stuff from skia_library_opts.gyp") 699 assert(false, "Need to port cpu specific stuff from skia_library_opts.gyp")
700 } 700 }
701 701
702 if (is_android && !is_debug) { 702 if ((is_android || is_win) && !is_debug) {
703 configs -= [ "//build/config/compiler:default_optimization" ] 703 configs -= [ "//build/config/compiler:default_optimization" ]
704 configs += [ "//build/config/compiler:optimize_max" ] 704 configs += [ "//build/config/compiler:optimize_max" ]
705 } 705 }
706 706
707 configs -= [ "//build/config/compiler:chromium_code" ] 707 configs -= [ "//build/config/compiler:chromium_code" ]
708 configs += [ 708 configs += [
709 ":skia_config", 709 ":skia_config",
710 ":skia_library_config", 710 ":skia_library_config",
711 "//build/config/compiler:no_chromium_code", 711 "//build/config/compiler:no_chromium_code",
712 ] 712 ]
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
773 773
774 deps = [ 774 deps = [
775 ":skia", 775 ":skia",
776 "//base", 776 "//base",
777 "//base/test:test_support", 777 "//base/test:test_support",
778 "//build/config/sanitizers:deps", 778 "//build/config/sanitizers:deps",
779 "//build/win:default_exe_manifest", 779 "//build/win:default_exe_manifest",
780 ] 780 ]
781 } 781 }
782 } 782 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698