blob: 2345ebc250a03520050e4113d16fce5936988e62 [file] [log] [blame]
Colin Crossb98c8b02016-07-29 13:44:28 -07001package config
Colin Cross3f40fa42015-01-30 17:27:36 -08002
3import (
4 "sort"
5 "strings"
6)
7
8// Cflags that should be filtered out when compiling with clang
Colin Crossb98c8b02016-07-29 13:44:28 -07009var ClangUnknownCflags = sorted([]string{
Colin Cross3f40fa42015-01-30 17:27:36 -080010 "-finline-functions",
11 "-finline-limit=64",
12 "-fno-canonical-system-headers",
Dan Willemsen3bf6b472015-09-11 17:41:10 -070013 "-Wno-clobbered",
14 "-fno-devirtualize",
Colin Cross3f40fa42015-01-30 17:27:36 -080015 "-fno-tree-sra",
Colin Crossa360e8b2015-03-16 16:22:28 -070016 "-fprefetch-loop-arrays",
Colin Cross3f40fa42015-01-30 17:27:36 -080017 "-funswitch-loops",
Dan Willemsene8c52372016-05-19 16:57:11 -070018 "-Werror=unused-but-set-parameter",
19 "-Werror=unused-but-set-variable",
Colin Cross3f40fa42015-01-30 17:27:36 -080020 "-Wmaybe-uninitialized",
Dan Willemsen3bf6b472015-09-11 17:41:10 -070021 "-Wno-error=clobbered",
Colin Cross3f40fa42015-01-30 17:27:36 -080022 "-Wno-error=maybe-uninitialized",
Colin Cross74d1ec02015-04-28 13:30:13 -070023 "-Wno-error=unused-but-set-parameter",
24 "-Wno-error=unused-but-set-variable",
Colin Cross3f40fa42015-01-30 17:27:36 -080025 "-Wno-free-nonheap-object",
26 "-Wno-literal-suffix",
27 "-Wno-maybe-uninitialized",
28 "-Wno-old-style-declaration",
29 "-Wno-psabi",
Colin Cross3f40fa42015-01-30 17:27:36 -080030 "-Wno-unused-but-set-parameter",
Colin Cross74d1ec02015-04-28 13:30:13 -070031 "-Wno-unused-but-set-variable",
Colin Cross3f40fa42015-01-30 17:27:36 -080032 "-Wno-unused-local-typedefs",
Colin Cross62ec5f42015-03-18 17:20:28 -070033 "-Wunused-but-set-parameter",
Colin Cross74d1ec02015-04-28 13:30:13 -070034 "-Wunused-but-set-variable",
Dan Willemsene6540452015-10-20 15:21:33 -070035 "-fdiagnostics-color",
Colin Cross3f40fa42015-01-30 17:27:36 -080036
37 // arm + arm64 + mips + mips64
38 "-fgcse-after-reload",
39 "-frerun-cse-after-loop",
40 "-frename-registers",
41 "-fno-strict-volatile-bitfields",
42
43 // arm + arm64
44 "-fno-align-jumps",
Colin Cross3f40fa42015-01-30 17:27:36 -080045
46 // arm
47 "-mthumb-interwork",
48 "-fno-builtin-sin",
49 "-fno-caller-saves",
50 "-fno-early-inlining",
51 "-fno-move-loop-invariants",
52 "-fno-partial-inlining",
53 "-fno-tree-copy-prop",
54 "-fno-tree-loop-optimize",
55
56 // mips + mips64
57 "-msynci",
Dan Willemsen3bf6b472015-09-11 17:41:10 -070058 "-mno-synci",
Colin Cross3f40fa42015-01-30 17:27:36 -080059 "-mno-fused-madd",
60
61 // x86 + x86_64
62 "-finline-limit=300",
63 "-fno-inline-functions-called-once",
64 "-mfpmath=sse",
65 "-mbionic",
Dan Willemsene6540452015-10-20 15:21:33 -070066})
Colin Cross3f40fa42015-01-30 17:27:36 -080067
Jayant Chowdhary9677e8c2017-06-15 14:45:18 -070068var ClangLibToolingUnknownCflags = []string{
Jayant Chowdhary20f5d132017-10-27 11:29:37 -070069 "-flto*",
Jayant Chowdhary9677e8c2017-06-15 14:45:18 -070070 "-fsanitize*",
71}
72
Colin Cross3f40fa42015-01-30 17:27:36 -080073func init() {
Colin Crossb98c8b02016-07-29 13:44:28 -070074 pctx.StaticVariable("ClangExtraCflags", strings.Join([]string{
Colin Cross3f40fa42015-01-30 17:27:36 -080075 "-D__compiler_offsetof=__builtin_offsetof",
76
77 // Help catch common 32/64-bit errors.
78 "-Werror=int-conversion",
79
Colin Cross74d1ec02015-04-28 13:30:13 -070080 // Disable overly aggressive warning for macros defined with a leading underscore
81 // This happens in AndroidConfig.h, which is included nearly everywhere.
Dan Willemsen3bf6b472015-09-11 17:41:10 -070082 // TODO: can we remove this now?
Colin Cross74d1ec02015-04-28 13:30:13 -070083 "-Wno-reserved-id-macro",
84
85 // Disable overly aggressive warning for format strings.
86 // Bug: 20148343
87 "-Wno-format-pedantic",
88
Colin Cross3f40fa42015-01-30 17:27:36 -080089 // Workaround for ccache with clang.
90 // See http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html.
91 "-Wno-unused-command-line-argument",
92
Dan Willemsene6540452015-10-20 15:21:33 -070093 // Force clang to always output color diagnostics. Ninja will strip the ANSI
94 // color codes if it is not running in a terminal.
95 "-fcolor-diagnostics",
Pirama Arumuga Nainarb6572b12016-06-28 10:56:03 -070096
97 // http://b/29823425 Disable -Wexpansion-to-defined for Clang update to r271374
98 "-Wno-expansion-to-defined",
Dan Willemsen253cab82017-03-27 16:53:38 -070099
Stephen Hines0ed7d242017-10-04 16:12:37 -0700100 // http://b/68236239 Allow 0/NULL instead of using nullptr everywhere.
101 "-Wno-zero-as-null-pointer-constant",
102
Dan Willemsen253cab82017-03-27 16:53:38 -0700103 // http://b/36463318 Clang executes with an absolute path, so clang-provided
104 // headers are now absolute.
105 "-fdebug-prefix-map=$$PWD/=",
Colin Cross3f40fa42015-01-30 17:27:36 -0800106 }, " "))
107
Colin Crossb98c8b02016-07-29 13:44:28 -0700108 pctx.StaticVariable("ClangExtraCppflags", strings.Join([]string{
Dan Willemsenac5e1cb2016-01-12 16:22:40 -0800109 // Disable -Winconsistent-missing-override until we can clean up the existing
110 // codebase for it.
111 "-Wno-inconsistent-missing-override",
Pirama Arumuga Nainarb6572b12016-06-28 10:56:03 -0700112
113 // Bug: http://b/29823425 Disable -Wnull-dereference until the
114 // new instances detected by this warning are fixed.
115 "-Wno-null-dereference",
Josh Gaoe0b933b2017-04-26 20:26:14 -0700116
117 // Enable clang's thread-safety annotations in libcxx.
118 // Turn off -Wthread-safety-negative, to avoid breaking projects that use -Weverything.
119 "-D_LIBCPP_ENABLE_THREAD_SAFETY_ANNOTATIONS",
120 "-Wno-thread-safety-negative",
Dan Willemsenac5e1cb2016-01-12 16:22:40 -0800121 }, " "))
122
Colin Crossb98c8b02016-07-29 13:44:28 -0700123 pctx.StaticVariable("ClangExtraTargetCflags", strings.Join([]string{
Colin Cross3f40fa42015-01-30 17:27:36 -0800124 "-nostdlibinc",
125 }, " "))
Dan Willemsenbe03f342016-03-03 17:21:04 -0800126
Colin Crossb98c8b02016-07-29 13:44:28 -0700127 pctx.StaticVariable("ClangExtraNoOverrideCflags", strings.Join([]string{
Dan Willemsenbe03f342016-03-03 17:21:04 -0800128 "-Werror=address-of-temporary",
Pirama Arumuga Nainarb6572b12016-06-28 10:56:03 -0700129 // Bug: http://b/29823425 Disable -Wnull-dereference until the
130 // new cases detected by this warning in Clang r271374 are
131 // fixed.
132 //"-Werror=null-dereference",
Dan Willemsenbe03f342016-03-03 17:21:04 -0800133 "-Werror=return-type",
134 }, " "))
Colin Cross3f40fa42015-01-30 17:27:36 -0800135}
136
Colin Crossb98c8b02016-07-29 13:44:28 -0700137func ClangFilterUnknownCflags(cflags []string) []string {
Colin Cross3f40fa42015-01-30 17:27:36 -0800138 ret := make([]string, 0, len(cflags))
139 for _, f := range cflags {
Colin Crossb98c8b02016-07-29 13:44:28 -0700140 if !inListSorted(f, ClangUnknownCflags) {
Colin Cross3f40fa42015-01-30 17:27:36 -0800141 ret = append(ret, f)
142 }
143 }
144
145 return ret
146}
147
148func inListSorted(s string, list []string) bool {
149 for _, l := range list {
150 if s == l {
151 return true
152 } else if s < l {
153 return false
154 }
155 }
156 return false
157}
Dan Willemsene6540452015-10-20 15:21:33 -0700158
159func sorted(list []string) []string {
160 sort.Strings(list)
161 return list
162}