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

Side by Side Diff: third_party/expat/BUILD.gn

Issue 2897283002: Pass -Oz flag to Clang when targeting Android. (Closed)
Patch Set: Created 3 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 unified diff | Download patch
« no previous file with comments | « build/config/compiler/BUILD.gn ('k') | 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/chromecast_build.gni") 5 import("//build/config/chromecast_build.gni")
6 import("//testing/libfuzzer/fuzzer_test.gni") 6 import("//testing/libfuzzer/fuzzer_test.gni")
7 7
8 # On Linux, we implicitly already depend on expat via fontconfig; 8 # On Linux, we implicitly already depend on expat via fontconfig;
9 # let's not pull it in twice. 9 # let's not pull it in twice.
10 # Chromecast doesn't ship expat as a system library. 10 # Chromecast doesn't ship expat as a system library.
(...skipping 15 matching lines...) Expand all
26 static_library("expat") { 26 static_library("expat") {
27 sources = [ 27 sources = [
28 "files/lib/expat.h", 28 "files/lib/expat.h",
29 "files/lib/xmlparse.c", 29 "files/lib/xmlparse.c",
30 "files/lib/xmlrole.c", 30 "files/lib/xmlrole.c",
31 "files/lib/xmltok.c", 31 "files/lib/xmltok.c",
32 ] 32 ]
33 33
34 public_configs = [ ":expat_config" ] 34 public_configs = [ ":expat_config" ]
35 35
36 # TODO(thakis): Remove this once clang no longer crashes when building
37 # libexpat with -Oz.
Nico 2017/05/24 15:33:35 (I checked that this no longer happens with trunk
38 configs -= [ "//build/config/compiler:default_optimization" ]
39 configs += [ "//build/config/compiler:optimize_max" ]
40
36 defines = [ "_LIB" ] 41 defines = [ "_LIB" ]
37 if (is_win) { 42 if (is_win) {
38 defines += [ "COMPILED_FROM_DSP" ] 43 defines += [ "COMPILED_FROM_DSP" ]
39 } else { 44 } else {
40 defines += [ "HAVE_EXPAT_CONFIG_H" ] 45 defines += [ "HAVE_EXPAT_CONFIG_H" ]
41 } 46 }
42 } 47 }
43 } 48 }
44 49
45 fuzzer_test("expat_xml_parse_fuzzer") { 50 fuzzer_test("expat_xml_parse_fuzzer") {
46 sources = [ 51 sources = [
47 "fuzz/expat_xml_parse_fuzzer.cc", 52 "fuzz/expat_xml_parse_fuzzer.cc",
48 ] 53 ]
49 deps = [ 54 deps = [
50 ":expat", 55 ":expat",
51 ] 56 ]
52 dict = "//testing/libfuzzer/fuzzers/dicts/xml.dict" 57 dict = "//testing/libfuzzer/fuzzers/dicts/xml.dict"
53 } 58 }
OLDNEW
« no previous file with comments | « build/config/compiler/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698