Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 1 | // Copyright 2016 Google Inc. All rights reserved. |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | // you may not use this file except in compliance with the License. |
| 5 | // You may obtain a copy of the License at |
| 6 | // |
| 7 | // http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | // See the License for the specific language governing permissions and |
| 13 | // limitations under the License. |
| 14 | |
Colin Cross | b98c8b0 | 2016-07-29 13:44:28 -0700 | [diff] [blame] | 15 | package config |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 16 | |
| 17 | import ( |
Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 18 | "runtime" |
Colin Cross | bcf5370 | 2024-01-17 11:11:03 -0800 | [diff] [blame] | 19 | "slices" |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 20 | "strings" |
| 21 | |
| 22 | "android/soong/android" |
Ramy Medhat | 9a90fe5 | 2020-04-13 13:21:23 -0400 | [diff] [blame] | 23 | "android/soong/remoteexec" |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 24 | ) |
| 25 | |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 26 | var ( |
Cole Faust | 8982b1c | 2024-04-08 16:54:45 -0700 | [diff] [blame] | 27 | pctx = android.NewPackageContext("android/soong/cc/config") |
Sam Delmerico | 7f88956 | 2022-03-25 14:55:40 +0000 | [diff] [blame] | 28 | |
Leo Li | 8756b37 | 2017-05-22 16:11:34 -0700 | [diff] [blame] | 29 | // Flags used by lots of devices. Putting them in package static variables |
| 30 | // will save bytes in build.ninja so they aren't repeated for every file |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 31 | commonGlobalCflags = []string{ |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 32 | // Enable some optimization by default. |
| 33 | "-O2", |
| 34 | |
| 35 | // Warnings enabled by default. Reference: |
| 36 | // https://clang.llvm.org/docs/DiagnosticsReference.html |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 37 | "-Wall", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 38 | "-Wextra", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 39 | "-Winit-self", |
| 40 | "-Wpointer-arith", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 41 | "-Wunguarded-availability", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 42 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 43 | // Warnings treated as errors by default. |
| 44 | // See also noOverrideGlobalCflags for errors that cannot be disabled |
| 45 | // from Android.bp files. |
Colin Cross | 7278afc | 2017-11-02 22:38:32 -0700 | [diff] [blame] | 46 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 47 | // Using __DATE__/__TIME__ causes build nondeterminism. |
Dan Willemsen | 5d980c8 | 2019-08-27 19:37:10 -0700 | [diff] [blame] | 48 | "-Werror=date-time", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 49 | // Detects forgotten */& that usually cause a crash |
| 50 | "-Werror=int-conversion", |
| 51 | // Detects unterminated alignment modification pragmas, which often lead |
| 52 | // to ABI mismatch between modules and hard-to-debug crashes. |
Elliott Hughes | 2cdbdf1 | 2019-12-03 18:13:00 -0800 | [diff] [blame] | 53 | "-Werror=pragma-pack", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 54 | // Same as above, but detects alignment pragmas around a header |
| 55 | // inclusion. |
Elliott Hughes | 2cdbdf1 | 2019-12-03 18:13:00 -0800 | [diff] [blame] | 56 | "-Werror=pragma-pack-suspicious-include", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 57 | // Detects dividing an array size by itself, which is a common typo that |
| 58 | // leads to bugs. |
| 59 | "-Werror=sizeof-array-div", |
| 60 | // Detects a typo that cuts off a prefix from a string literal. |
Christopher Di Bella | 23a991c | 2020-11-11 22:41:32 +0000 | [diff] [blame] | 61 | "-Werror=string-plus-int", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 62 | // Detects for loops that will never execute more than once (for example |
| 63 | // due to unconditional break), but have a non-empty loop increment |
| 64 | // clause. Often a mistake/bug. |
George Burgess IV | fb81db2 | 2020-02-22 19:28:56 -0800 | [diff] [blame] | 65 | "-Werror=unreachable-code-loop-increment", |
Colin Cross | 9cc59c1 | 2021-07-14 17:59:54 -0700 | [diff] [blame] | 66 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 67 | // Warnings that should not be errors even for modules with -Werror. |
Krzysztof Kosiński | 6934b0e | 2022-08-07 06:56:54 +0000 | [diff] [blame] | 68 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 69 | // Making deprecated usages an error causes extreme pain when trying to |
| 70 | // deprecate anything. |
| 71 | "-Wno-error=deprecated-declarations", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 72 | |
| 73 | // Warnings disabled by default. |
| 74 | |
Elliott Hughes | afd8aac | 2024-10-31 21:42:40 +0000 | [diff] [blame] | 75 | // We should encourage use of C23 features even when the whole project |
| 76 | // isn't C23-ready. |
| 77 | "-Wno-c23-extensions", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 78 | // Designated initializer syntax is recommended by the Google C++ style |
| 79 | // and is OK to use even if not formally supported by the chosen C++ |
| 80 | // version. |
| 81 | "-Wno-c99-designator", |
| 82 | // Detects uses of a GNU C extension equivalent to a limited form of |
| 83 | // constexpr. Enabling this would require replacing many constants with |
| 84 | // macros, which is not a good trade-off. |
| 85 | "-Wno-gnu-folding-constant", |
| 86 | // AIDL generated code redeclares pure virtual methods in each |
| 87 | // subsequent version of an interface, so this warning is currently |
| 88 | // infeasible to enable. |
| 89 | "-Wno-inconsistent-missing-override", |
Krzysztof Kosiński | 8f8cc16 | 2023-10-20 01:02:58 +0000 | [diff] [blame] | 90 | // Detects designated initializers that are in a different order than |
| 91 | // the fields in the initialized type, which causes the side effects |
| 92 | // of initializers to occur out of order with the source code. |
| 93 | // In practice, this warning has extremely poor signal to noise ratio, |
| 94 | // because it is triggered even for initializers with no side effects. |
| 95 | // Individual modules can still opt into it via cflags. |
| 96 | "-Wno-error=reorder-init-list", |
| 97 | "-Wno-reorder-init-list", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 98 | // Incompatible with the Google C++ style guidance to use 'int' for loop |
| 99 | // indices; poor signal to noise ratio. |
| 100 | "-Wno-sign-compare", |
| 101 | // Poor signal to noise ratio. |
| 102 | "-Wno-unused", |
| 103 | |
| 104 | // Global preprocessor constants. |
| 105 | |
| 106 | "-DANDROID", |
| 107 | "-DNDEBUG", |
| 108 | "-UDEBUG", |
Colin Cross | 9cc59c1 | 2021-07-14 17:59:54 -0700 | [diff] [blame] | 109 | "-D__compiler_offsetof=__builtin_offsetof", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 110 | // Allows the bionic versioning.h to indirectly determine whether the |
| 111 | // option -Wunguarded-availability is on or not. |
| 112 | "-D__ANDROID_UNAVAILABLE_SYMBOLS_ARE_WEAK__", |
| 113 | |
| 114 | // -f and -g options. |
Colin Cross | 9cc59c1 | 2021-07-14 17:59:54 -0700 | [diff] [blame] | 115 | |
| 116 | // Emit address-significance table which allows linker to perform safe ICF. Clang does |
| 117 | // not emit the table by default on Android since NDK still uses GNU binutils. |
| 118 | "-faddrsig", |
| 119 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 120 | // Emit debugging data in a modern format (DWARF v5). |
| 121 | "-fdebug-default-version=5", |
Colin Cross | 9cc59c1 | 2021-07-14 17:59:54 -0700 | [diff] [blame] | 122 | |
Colin Cross | 9cc59c1 | 2021-07-14 17:59:54 -0700 | [diff] [blame] | 123 | // Force clang to always output color diagnostics. Ninja will strip the ANSI |
| 124 | // color codes if it is not running in a terminal. |
| 125 | "-fcolor-diagnostics", |
| 126 | |
Pirama Arumuga Nainar | 8a4804f | 2022-02-14 11:19:26 -0800 | [diff] [blame] | 127 | // Turn off FMA which got enabled by default in clang-r445002 (http://b/218805949) |
| 128 | "-ffp-contract=off", |
AdityaK | 423e4ce | 2023-05-26 12:08:05 -0700 | [diff] [blame] | 129 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 130 | // Google C++ style does not allow exceptions, turn them off by default. |
| 131 | "-fno-exceptions", |
| 132 | |
| 133 | // Disable optimizations based on strict aliasing by default. |
| 134 | // The performance benefit of enabling them currently does not outweigh |
| 135 | // the risk of hard-to-reproduce bugs. |
| 136 | "-fno-strict-aliasing", |
| 137 | |
| 138 | // Disable line wrapping for error messages - it interferes with |
| 139 | // displaying logs in web browsers. |
| 140 | "-fmessage-length=0", |
| 141 | |
AdityaK | 423e4ce | 2023-05-26 12:08:05 -0700 | [diff] [blame] | 142 | // Using simple template names reduces the size of debug builds. |
| 143 | "-gsimple-template-names", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 144 | |
Eric Rahm | 5571ab8 | 2023-10-20 15:56:17 +0000 | [diff] [blame] | 145 | // Use zstd to compress debug data. |
| 146 | "-gz=zstd", |
| 147 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 148 | // Make paths in deps files relative. |
| 149 | "-no-canonical-prefixes", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 150 | } |
| 151 | |
Colin Cross | 6f6a428 | 2016-10-17 14:19:06 -0700 | [diff] [blame] | 152 | commonGlobalConlyflags = []string{} |
Elliott Hughes | 5a0401a | 2016-10-07 13:12:58 -0700 | [diff] [blame] | 153 | |
Liz Kammer | e4d1bda | 2022-06-22 21:02:08 +0000 | [diff] [blame] | 154 | commonGlobalAsflags = []string{ |
| 155 | "-D__ASSEMBLY__", |
| 156 | // TODO(b/235105792): override global -fdebug-default-version=5, it is causing $TMPDIR to |
| 157 | // end up in the dwarf data for crtend_so.S. |
| 158 | "-fdebug-default-version=4", |
| 159 | } |
| 160 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 161 | // Compilation flags for device code; not applied to host code. |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 162 | deviceGlobalCflags = []string{ |
Colin Cross | 133dbe7 | 2017-11-02 22:55:19 -0700 | [diff] [blame] | 163 | "-ffunction-sections", |
Colin Cross | ea3141d | 2017-11-06 14:02:02 -0800 | [diff] [blame] | 164 | "-fdata-sections", |
| 165 | "-fno-short-enums", |
Colin Cross | 133dbe7 | 2017-11-02 22:55:19 -0700 | [diff] [blame] | 166 | "-funwind-tables", |
| 167 | "-fstack-protector-strong", |
| 168 | "-Wa,--noexecstack", |
| 169 | "-D_FORTIFY_SOURCE=2", |
| 170 | |
| 171 | "-Wstrict-aliasing=2", |
| 172 | |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 173 | "-Werror=return-type", |
| 174 | "-Werror=non-virtual-dtor", |
| 175 | "-Werror=address", |
| 176 | "-Werror=sequence-point", |
Colin Cross | 133dbe7 | 2017-11-02 22:55:19 -0700 | [diff] [blame] | 177 | "-Werror=format-security", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 178 | } |
| 179 | |
Yi Kong | a9e1df1 | 2022-10-20 14:45:52 +0900 | [diff] [blame] | 180 | commonGlobalLldflags = []string{ |
| 181 | "-fuse-ld=lld", |
| 182 | "-Wl,--icf=safe", |
Steven Moreland | 8fe37e4 | 2023-07-22 00:14:55 +0000 | [diff] [blame] | 183 | "-Wl,--no-demangle", |
Yi Kong | a9e1df1 | 2022-10-20 14:45:52 +0900 | [diff] [blame] | 184 | } |
| 185 | |
Colin Cross | 26f1450 | 2017-11-06 13:59:48 -0800 | [diff] [blame] | 186 | deviceGlobalCppflags = []string{ |
| 187 | "-fvisibility-inlines-hidden", |
| 188 | } |
| 189 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 190 | // Linking flags for device code; not applied to host binaries. |
Colin Cross | 324a457 | 2017-11-02 23:09:41 -0700 | [diff] [blame] | 191 | deviceGlobalLdflags = []string{ |
| 192 | "-Wl,-z,noexecstack", |
| 193 | "-Wl,-z,relro", |
| 194 | "-Wl,-z,now", |
| 195 | "-Wl,--build-id=md5", |
Colin Cross | 324a457 | 2017-11-02 23:09:41 -0700 | [diff] [blame] | 196 | "-Wl,--fatal-warnings", |
| 197 | "-Wl,--no-undefined-version", |
Ryan Prichard | b35a85e | 2021-01-13 19:18:53 -0800 | [diff] [blame] | 198 | // TODO: Eventually we should link against a libunwind.a with hidden symbols, and then these |
| 199 | // --exclude-libs arguments can be removed. |
Christopher Ferris | c3a1e22 | 2019-04-10 17:57:50 -0700 | [diff] [blame] | 200 | "-Wl,--exclude-libs,libgcc.a", |
Yi Kong | 3d8792f | 2019-05-06 16:18:33 -0700 | [diff] [blame] | 201 | "-Wl,--exclude-libs,libgcc_stripped.a", |
Peter Collingbourne | e5ba286 | 2019-12-10 18:37:45 -0800 | [diff] [blame] | 202 | "-Wl,--exclude-libs,libunwind_llvm.a", |
Ryan Prichard | b35a85e | 2021-01-13 19:18:53 -0800 | [diff] [blame] | 203 | "-Wl,--exclude-libs,libunwind.a", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 204 | } |
| 205 | |
Eric Rahm | 1952471 | 2023-10-20 15:56:27 +0000 | [diff] [blame] | 206 | deviceGlobalLldflags = append(append(deviceGlobalLdflags, commonGlobalLldflags...), |
| 207 | "-Wl,--compress-debug-sections=zstd", |
| 208 | ) |
Chih-Hung Hsieh | 02b4da5 | 2018-04-03 11:33:34 -0700 | [diff] [blame] | 209 | |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 210 | hostGlobalCflags = []string{} |
| 211 | |
Colin Cross | 26f1450 | 2017-11-06 13:59:48 -0800 | [diff] [blame] | 212 | hostGlobalCppflags = []string{} |
| 213 | |
Colin Cross | 324a457 | 2017-11-02 23:09:41 -0700 | [diff] [blame] | 214 | hostGlobalLdflags = []string{} |
| 215 | |
Yi Kong | a9e1df1 | 2022-10-20 14:45:52 +0900 | [diff] [blame] | 216 | hostGlobalLldflags = commonGlobalLldflags |
Chih-Hung Hsieh | 02b4da5 | 2018-04-03 11:33:34 -0700 | [diff] [blame] | 217 | |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 218 | commonGlobalCppflags = []string{ |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 219 | // -Wimplicit-fallthrough is not enabled by -Wall. |
| 220 | "-Wimplicit-fallthrough", |
| 221 | |
| 222 | // Enable clang's thread-safety annotations in libcxx. |
| 223 | "-D_LIBCPP_ENABLE_THREAD_SAFETY_ANNOTATIONS", |
| 224 | |
| 225 | // libc++'s math.h has an #include_next outside of system_headers. |
| 226 | "-Wno-gnu-include-next", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 227 | } |
| 228 | |
Krzysztof Kosiński | 982c588 | 2023-08-18 18:56:45 +0000 | [diff] [blame] | 229 | // These flags are appended after the module's cflags, so they cannot be |
| 230 | // overridden from Android.bp files. |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 231 | // |
| 232 | // NOTE: if you need to disable a warning to unblock a compiler upgrade |
| 233 | // and it is only triggered by third party code, add it to |
| 234 | // extraExternalCflags (if possible) or noOverrideExternalGlobalCflags |
| 235 | // (if the former doesn't work). If the new warning also occurs in first |
| 236 | // party code, try adding it to commonGlobalCflags first. Adding it here |
| 237 | // should be the last resort, because it prevents all code in Android from |
| 238 | // opting into the warning. |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 239 | noOverrideGlobalCflags = []string{ |
Christopher Di Bella | 23a991c | 2020-11-11 22:41:32 +0000 | [diff] [blame] | 240 | "-Werror=bool-operation", |
Zijun | 7436834 | 2024-11-26 07:58:35 +0000 | [diff] [blame] | 241 | "-Werror=dangling", |
Zijun Zhao | d156908 | 2023-03-08 23:48:45 +0000 | [diff] [blame] | 242 | "-Werror=format-insufficient-args", |
Christopher Di Bella | 23a991c | 2020-11-11 22:41:32 +0000 | [diff] [blame] | 243 | "-Werror=implicit-int-float-conversion", |
| 244 | "-Werror=int-in-bool-context", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 245 | "-Werror=int-to-pointer-cast", |
| 246 | "-Werror=pointer-to-int-cast", |
Christopher Di Bella | 23a991c | 2020-11-11 22:41:32 +0000 | [diff] [blame] | 247 | "-Werror=xor-used-as-pow", |
Zijun | de4f1bf | 2024-11-27 19:52:37 +0000 | [diff] [blame] | 248 | "-Wimplicit-int-float-conversion", |
Stephen Hines | 2210e72 | 2020-07-15 11:11:57 -0700 | [diff] [blame] | 249 | // http://b/161386391 for -Wno-void-pointer-to-enum-cast |
| 250 | "-Wno-void-pointer-to-enum-cast", |
| 251 | // http://b/161386391 for -Wno-void-pointer-to-int-cast |
| 252 | "-Wno-void-pointer-to-int-cast", |
| 253 | // http://b/161386391 for -Wno-pointer-to-int-cast |
| 254 | "-Wno-pointer-to-int-cast", |
George Burgess IV | 6c69164 | 2019-09-18 17:52:05 -0700 | [diff] [blame] | 255 | "-Werror=fortify-source", |
Aditya Kumar | ef7c121 | 2024-02-02 17:22:03 +0000 | [diff] [blame] | 256 | // http://b/315246135 temporarily disabled |
AdityaK | 94688b5 | 2024-02-08 13:50:18 -0800 | [diff] [blame] | 257 | "-Wno-unused-variable", |
Aditya Kumar | ef7c121 | 2024-02-02 17:22:03 +0000 | [diff] [blame] | 258 | // Disabled because it produces many false positives. http://b/323050926 |
| 259 | "-Wno-missing-field-initializers", |
| 260 | // http://b/323050889 |
| 261 | "-Wno-packed-non-pod", |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 262 | |
| 263 | "-Werror=address-of-temporary", |
Krzysztof Kosiński | 83199b5 | 2023-10-26 07:16:29 +0000 | [diff] [blame] | 264 | "-Werror=incompatible-function-pointer-types", |
zijunzhao | 2863c0a | 2023-02-06 21:28:30 +0000 | [diff] [blame] | 265 | "-Werror=null-dereference", |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 266 | "-Werror=return-type", |
| 267 | |
| 268 | // http://b/72331526 Disable -Wtautological-* until the instances detected by these |
| 269 | // new warnings are fixed. |
| 270 | "-Wno-tautological-constant-compare", |
| 271 | "-Wno-tautological-type-limit-compare", |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 272 | // http://b/145211066 |
| 273 | "-Wno-implicit-int-float-conversion", |
| 274 | // New warnings to be fixed after clang-r377782. |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 275 | "-Wno-tautological-overlap-compare", // http://b/148815696 |
| 276 | // New warnings to be fixed after clang-r383902. |
| 277 | "-Wno-deprecated-copy", // http://b/153746672 |
| 278 | "-Wno-range-loop-construct", // http://b/153747076 |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 279 | "-Wno-zero-as-null-pointer-constant", // http://b/68236239 |
| 280 | "-Wno-deprecated-anon-enum-enum-conversion", // http://b/153746485 |
Elliott Hughes | 2ced4b5 | 2024-01-22 15:27:24 -0800 | [diff] [blame] | 281 | "-Wno-deprecated-enum-enum-conversion", |
Yi Kong | fd50d17 | 2024-08-12 14:41:09 +0800 | [diff] [blame] | 282 | "-Wno-error=pessimizing-move", // http://b/154270751 |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 283 | // New warnings to be fixed after clang-r399163 |
| 284 | "-Wno-non-c-typedef-for-linkage", // http://b/161304145 |
Yabin Cui | 10bf3b8 | 2021-08-10 15:42:10 +0000 | [diff] [blame] | 285 | // New warnings to be fixed after clang-r428724 |
| 286 | "-Wno-align-mismatch", // http://b/193679946 |
Yi Kong | e827329 | 2021-08-31 14:04:18 +0800 | [diff] [blame] | 287 | // New warnings to be fixed after clang-r433403 |
| 288 | "-Wno-error=unused-but-set-variable", // http://b/197240255 |
| 289 | "-Wno-error=unused-but-set-parameter", // http://b/197240255 |
Chih-hung Hsieh | 5c40a92 | 2022-09-27 07:19:03 +0000 | [diff] [blame] | 290 | // New warnings to be fixed after clang-r468909 |
Chih-hung Hsieh | 5c40a92 | 2022-09-27 07:19:03 +0000 | [diff] [blame] | 291 | "-Wno-error=deprecated-builtins", // http://b/241601211 |
| 292 | "-Wno-error=deprecated", // in external/googletest/googletest |
Aditya Kumar | 2c6b4ac | 2024-09-12 18:40:17 +0000 | [diff] [blame] | 293 | // Disabling until the warning is fixed in libc++abi header files b/366180429 |
| 294 | "-Wno-deprecated-dynamic-exception-spec", |
Chris Wailes | ffebc5b | 2024-06-10 18:06:52 +0000 | [diff] [blame] | 295 | // New warnings to be fixed after clang-r522817 |
| 296 | "-Wno-error=invalid-offsetof", |
| 297 | "-Wno-error=thread-safety-reference-return", |
Elliott Hughes | 2ced4b5 | 2024-01-22 15:27:24 -0800 | [diff] [blame] | 298 | |
Chris Wailes | ffebc5b | 2024-06-10 18:06:52 +0000 | [diff] [blame] | 299 | // Allow using VLA CXX extension. |
| 300 | "-Wno-vla-cxx-extension", |
Aditya Kumar | 0654016 | 2024-09-05 10:36:49 -0700 | [diff] [blame] | 301 | "-Wno-cast-function-type-mismatch", |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 302 | } |
| 303 | |
zijunzhao | 933e380 | 2023-01-12 07:26:20 +0000 | [diff] [blame] | 304 | noOverride64GlobalCflags = []string{} |
| 305 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 306 | // Extra cflags applied to third-party code (anything for which |
| 307 | // IsThirdPartyPath() in build/soong/android/paths.go returns true; |
| 308 | // includes external/, most of vendor/ and most of hardware/) |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 309 | extraExternalCflags = []string{ |
| 310 | "-Wno-enum-compare", |
| 311 | "-Wno-enum-compare-switch", |
| 312 | |
| 313 | // http://b/72331524 Allow null pointer arithmetic until the instances detected by |
| 314 | // this new warning are fixed. |
| 315 | "-Wno-null-pointer-arithmetic", |
| 316 | |
| 317 | // Bug: http://b/29823425 Disable -Wnull-dereference until the |
| 318 | // new instances detected by this warning are fixed. |
| 319 | "-Wno-null-dereference", |
| 320 | |
| 321 | // http://b/145211477 |
| 322 | "-Wno-pointer-compare", |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 323 | "-Wno-final-dtor-non-final-class", |
| 324 | |
| 325 | // http://b/165945989 |
| 326 | "-Wno-psabi", |
Yi Kong | e827329 | 2021-08-31 14:04:18 +0800 | [diff] [blame] | 327 | |
| 328 | // http://b/199369603 |
| 329 | "-Wno-null-pointer-subtraction", |
Yi Kong | 9feb029 | 2021-12-15 13:20:27 +0800 | [diff] [blame] | 330 | |
| 331 | // http://b/175068488 |
| 332 | "-Wno-string-concatenation", |
Yi Kong | eb8d04e | 2022-07-08 13:47:09 +0800 | [diff] [blame] | 333 | |
| 334 | // http://b/239661264 |
| 335 | "-Wno-deprecated-non-prototype", |
Elliott Hughes | 2ced4b5 | 2024-01-22 15:27:24 -0800 | [diff] [blame] | 336 | |
| 337 | "-Wno-unused", |
| 338 | "-Wno-deprecated", |
Yi Kong | 2e2d753 | 2024-08-09 14:56:58 +0900 | [diff] [blame] | 339 | |
| 340 | // http://b/315250603 temporarily disabled |
| 341 | "-Wno-error=format", |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 342 | } |
| 343 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 344 | // Similar to noOverrideGlobalCflags, but applies only to third-party code |
| 345 | // (see extraExternalCflags). |
| 346 | // This section can unblock compiler upgrades when a third party module that |
| 347 | // enables -Werror and some group of warnings explicitly triggers newly |
| 348 | // added warnings. |
| 349 | noOverrideExternalGlobalCflags = []string{ |
| 350 | // http://b/151457797 |
| 351 | "-fcommon", |
| 352 | // http://b/191699019 |
| 353 | "-Wno-format-insufficient-args", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 354 | // http://b/296321508 |
| 355 | // Introduced in response to a critical security vulnerability and |
| 356 | // should be a hard error - it requires only whitespace changes to fix. |
| 357 | "-Wno-misleading-indentation", |
| 358 | // Triggered by old LLVM code in external/llvm. Likely not worth |
| 359 | // enabling since it's a cosmetic issue. |
| 360 | "-Wno-bitwise-instead-of-logical", |
| 361 | |
Prashanth Swaminathan | 13b0637 | 2024-01-22 19:41:52 -0800 | [diff] [blame] | 362 | "-Wno-unused", |
| 363 | "-Wno-unused-parameter", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 364 | "-Wno-unused-but-set-parameter", |
| 365 | "-Wno-unqualified-std-cast-call", |
| 366 | "-Wno-array-parameter", |
| 367 | "-Wno-gnu-offsetof-extensions", |
Yi Kong | fd50d17 | 2024-08-12 14:41:09 +0800 | [diff] [blame] | 368 | "-Wno-pessimizing-move", |
Priyanka Advani (xWF) | 435c4ed | 2024-09-05 18:24:53 +0000 | [diff] [blame] | 369 | // TODO: Enable this warning http://b/315245071 |
| 370 | "-Wno-fortify-source", |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 371 | } |
| 372 | |
Chih-Hung Hsieh | ec450d9 | 2022-08-02 12:03:50 -0700 | [diff] [blame] | 373 | llvmNextExtraCommonGlobalCflags = []string{ |
Yi Kong | a8b0bfb | 2022-12-26 16:02:50 +0900 | [diff] [blame] | 374 | // Do not report warnings when testing with the top of trunk LLVM. |
AdityaK | e7b6067 | 2023-10-27 11:26:00 -0700 | [diff] [blame] | 375 | "-Wno-everything", |
Chih-Hung Hsieh | ec450d9 | 2022-08-02 12:03:50 -0700 | [diff] [blame] | 376 | } |
Yi Kong | b79fc58 | 2022-07-13 14:50:33 +0800 | [diff] [blame] | 377 | |
Krzysztof Kosiński | 1a4572e | 2023-09-02 01:08:37 +0000 | [diff] [blame] | 378 | // Flags that must not appear in any command line. |
Colin Cross | b98c8b0 | 2016-07-29 13:44:28 -0700 | [diff] [blame] | 379 | IllegalFlags = []string{ |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 380 | "-w", |
| 381 | } |
Colin Cross | 6f6a428 | 2016-10-17 14:19:06 -0700 | [diff] [blame] | 382 | |
Elliott Hughes | fb294e3 | 2023-06-14 10:42:45 -0700 | [diff] [blame] | 383 | CStdVersion = "gnu17" |
Elliott Hughes | c79d9e3 | 2022-01-13 14:56:02 -0800 | [diff] [blame] | 384 | CppStdVersion = "gnu++20" |
Elliott Hughes | fb294e3 | 2023-06-14 10:42:45 -0700 | [diff] [blame] | 385 | ExperimentalCStdVersion = "gnu2x" |
Tomasz Wasilczyk | e149616 | 2023-11-15 11:46:26 -0800 | [diff] [blame] | 386 | ExperimentalCppStdVersion = "gnu++2b" |
Jayant Chowdhary | 6e8115a | 2017-05-09 10:21:52 -0700 | [diff] [blame] | 387 | |
Leo Li | 8756b37 | 2017-05-22 16:11:34 -0700 | [diff] [blame] | 388 | // prebuilts/clang default settings. |
| 389 | ClangDefaultBase = "prebuilts/clang/host" |
Aditya Kumar | 0654016 | 2024-09-05 10:36:49 -0700 | [diff] [blame] | 390 | ClangDefaultVersion = "clang-r536225" |
Yabin Cui | 8c69b78 | 2024-07-13 23:38:11 -0700 | [diff] [blame] | 391 | ClangDefaultShortVersion = "19" |
Chih-Hung Hsieh | 64a38dc | 2017-11-14 14:09:14 -0800 | [diff] [blame] | 392 | |
Chih-Hung Hsieh | 775edde | 2017-12-24 22:24:47 -0800 | [diff] [blame] | 393 | // Directories with warnings from Android.bp files. |
Chih-Hung Hsieh | 64a38dc | 2017-11-14 14:09:14 -0800 | [diff] [blame] | 394 | WarningAllowedProjects = []string{ |
Chih-Hung Hsieh | 64a38dc | 2017-11-14 14:09:14 -0800 | [diff] [blame] | 395 | "device/", |
Chih-Hung Hsieh | 64a38dc | 2017-11-14 14:09:14 -0800 | [diff] [blame] | 396 | "vendor/", |
| 397 | } |
Trevor Radcliffe | 391a25d | 2023-03-22 20:22:27 +0000 | [diff] [blame] | 398 | |
| 399 | VersionScriptFlagPrefix = "-Wl,--version-script," |
Trevor Radcliffe | a772d65 | 2023-04-11 13:41:17 +0000 | [diff] [blame] | 400 | |
| 401 | VisibilityHiddenFlag = "-fvisibility=hidden" |
| 402 | VisibilityDefaultFlag = "-fvisibility=default" |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 403 | ) |
| 404 | |
| 405 | func init() { |
Colin Cross | 0c66bc6 | 2021-07-20 09:47:41 -0700 | [diff] [blame] | 406 | if runtime.GOOS == "linux" { |
Kousik Kumar | d207cbe | 2020-10-20 05:52:49 +0000 | [diff] [blame] | 407 | commonGlobalCflags = append(commonGlobalCflags, "-fdebug-prefix-map=/proc/self/cwd=") |
| 408 | } |
| 409 | |
Cole Faust | 8982b1c | 2024-04-08 16:54:45 -0700 | [diff] [blame] | 410 | pctx.StaticVariable("CommonGlobalConlyflags", strings.Join(commonGlobalConlyflags, " ")) |
| 411 | pctx.StaticVariable("CommonGlobalAsflags", strings.Join(commonGlobalAsflags, " ")) |
| 412 | pctx.StaticVariable("DeviceGlobalCppflags", strings.Join(deviceGlobalCppflags, " ")) |
| 413 | pctx.StaticVariable("DeviceGlobalLdflags", strings.Join(deviceGlobalLdflags, " ")) |
| 414 | pctx.StaticVariable("DeviceGlobalLldflags", strings.Join(deviceGlobalLldflags, " ")) |
| 415 | pctx.StaticVariable("HostGlobalCppflags", strings.Join(hostGlobalCppflags, " ")) |
| 416 | pctx.StaticVariable("HostGlobalLdflags", strings.Join(hostGlobalLdflags, " ")) |
| 417 | pctx.StaticVariable("HostGlobalLldflags", strings.Join(hostGlobalLldflags, " ")) |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 418 | |
Colin Cross | 0523ba2 | 2021-07-14 18:45:05 -0700 | [diff] [blame] | 419 | pctx.VariableFunc("CommonGlobalCflags", func(ctx android.PackageVarContext) string { |
Colin Cross | bcf5370 | 2024-01-17 11:11:03 -0800 | [diff] [blame] | 420 | flags := slices.Clone(commonGlobalCflags) |
Stephen Hines | 66c8b44 | 2019-06-10 17:40:12 -0700 | [diff] [blame] | 421 | |
| 422 | // http://b/131390872 |
| 423 | // Automatically initialize any uninitialized stack variables. |
Stephen Hines | 5c873ac | 2020-05-14 00:59:09 +0000 | [diff] [blame] | 424 | // Prefer zero-init if multiple options are set. |
Stephen Hines | 66c8b44 | 2019-06-10 17:40:12 -0700 | [diff] [blame] | 425 | if ctx.Config().IsEnvTrue("AUTO_ZERO_INITIALIZE") { |
Pirama Arumuga Nainar | 4026558 | 2023-05-11 00:01:25 +0000 | [diff] [blame] | 426 | flags = append(flags, "-ftrivial-auto-var-init=zero") |
Stephen Hines | 66c8b44 | 2019-06-10 17:40:12 -0700 | [diff] [blame] | 427 | } else if ctx.Config().IsEnvTrue("AUTO_PATTERN_INITIALIZE") { |
| 428 | flags = append(flags, "-ftrivial-auto-var-init=pattern") |
Stephen Hines | 797e195 | 2020-01-28 14:43:11 -0800 | [diff] [blame] | 429 | } else if ctx.Config().IsEnvTrue("AUTO_UNINITIALIZE") { |
| 430 | flags = append(flags, "-ftrivial-auto-var-init=uninitialized") |
Stephen Hines | 0e1d5d8 | 2020-01-30 15:06:00 -0800 | [diff] [blame] | 431 | } else { |
Stephen Hines | 5c873ac | 2020-05-14 00:59:09 +0000 | [diff] [blame] | 432 | // Default to zero initialization. |
Pirama Arumuga Nainar | 4026558 | 2023-05-11 00:01:25 +0000 | [diff] [blame] | 433 | flags = append(flags, "-ftrivial-auto-var-init=zero") |
Stephen Hines | 66c8b44 | 2019-06-10 17:40:12 -0700 | [diff] [blame] | 434 | } |
Yi Kong | 62e75f5 | 2021-08-18 15:38:20 +0800 | [diff] [blame] | 435 | |
| 436 | // Workaround for ccache with clang. |
| 437 | // See http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html. |
| 438 | if ctx.Config().IsEnvTrue("USE_CCACHE") { |
| 439 | flags = append(flags, "-Wno-unused-command-line-argument") |
| 440 | } |
Yi Kong | b79fc58 | 2022-07-13 14:50:33 +0800 | [diff] [blame] | 441 | |
Yi Kong | f7f69e4 | 2022-07-21 15:49:05 +0800 | [diff] [blame] | 442 | if ctx.Config().IsEnvTrue("ALLOW_UNKNOWN_WARNING_OPTION") { |
| 443 | flags = append(flags, "-Wno-error=unknown-warning-option") |
| 444 | } |
| 445 | |
Fabián Cañas | bc10544 | 2023-06-30 17:53:06 +0000 | [diff] [blame] | 446 | switch ctx.Config().Getenv("CLANG_DEFAULT_DEBUG_LEVEL") { |
| 447 | case "debug_level_0": |
| 448 | flags = append(flags, "-g0") |
| 449 | case "debug_level_1": |
| 450 | flags = append(flags, "-g1") |
| 451 | case "debug_level_2": |
| 452 | flags = append(flags, "-g2") |
| 453 | case "debug_level_3": |
| 454 | flags = append(flags, "-g3") |
| 455 | case "debug_level_g": |
| 456 | flags = append(flags, "-g") |
| 457 | default: |
| 458 | flags = append(flags, "-g") |
| 459 | } |
| 460 | |
Stephen Hines | 66c8b44 | 2019-06-10 17:40:12 -0700 | [diff] [blame] | 461 | return strings.Join(flags, " ") |
| 462 | }) |
| 463 | |
Colin Cross | 0523ba2 | 2021-07-14 18:45:05 -0700 | [diff] [blame] | 464 | pctx.VariableFunc("DeviceGlobalCflags", func(ctx android.PackageVarContext) string { |
Colin Cross | c8bed31 | 2021-07-14 17:56:21 -0700 | [diff] [blame] | 465 | return strings.Join(deviceGlobalCflags, " ") |
Doug Horn | c32c6b0 | 2019-01-17 14:44:05 -0800 | [diff] [blame] | 466 | }) |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 467 | |
Yi Kong | 60a2010 | 2023-01-13 04:40:18 +0900 | [diff] [blame] | 468 | pctx.VariableFunc("NoOverrideGlobalCflags", func(ctx android.PackageVarContext) string { |
| 469 | flags := noOverrideGlobalCflags |
| 470 | if ctx.Config().IsEnvTrue("LLVM_NEXT") { |
| 471 | flags = append(noOverrideGlobalCflags, llvmNextExtraCommonGlobalCflags...) |
AdityaK | e7b6067 | 2023-10-27 11:26:00 -0700 | [diff] [blame] | 472 | IllegalFlags = []string{} // Don't fail build while testing a new compiler. |
zijunzhao | 933e380 | 2023-01-12 07:26:20 +0000 | [diff] [blame] | 473 | } |
| 474 | return strings.Join(flags, " ") |
| 475 | }) |
| 476 | |
Cole Faust | 8982b1c | 2024-04-08 16:54:45 -0700 | [diff] [blame] | 477 | pctx.StaticVariable("NoOverride64GlobalCflags", strings.Join(noOverride64GlobalCflags, " ")) |
| 478 | pctx.StaticVariable("HostGlobalCflags", strings.Join(hostGlobalCflags, " ")) |
| 479 | pctx.StaticVariable("NoOverrideExternalGlobalCflags", strings.Join(noOverrideExternalGlobalCflags, " ")) |
| 480 | pctx.StaticVariable("CommonGlobalCppflags", strings.Join(commonGlobalCppflags, " ")) |
| 481 | pctx.StaticVariable("ExternalCflags", strings.Join(extraExternalCflags, " ")) |
Trevor Radcliffe | a772d65 | 2023-04-11 13:41:17 +0000 | [diff] [blame] | 482 | |
Colin Cross | 1cfd89a | 2016-09-15 09:30:46 -0700 | [diff] [blame] | 483 | // Everything in these lists is a crime against abstraction and dependency tracking. |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 484 | // Do not add anything to this list. |
Jingwen Chen | 2199995 | 2021-05-19 05:53:51 +0000 | [diff] [blame] | 485 | commonGlobalIncludes := []string{ |
| 486 | "system/core/include", |
| 487 | "system/logging/liblog/include", |
| 488 | "system/media/audio/include", |
| 489 | "hardware/libhardware/include", |
| 490 | "hardware/libhardware_legacy/include", |
| 491 | "hardware/ril/include", |
| 492 | "frameworks/native/include", |
| 493 | "frameworks/native/opengl/include", |
| 494 | "frameworks/av/include", |
| 495 | } |
Jingwen Chen | 2199995 | 2021-05-19 05:53:51 +0000 | [diff] [blame] | 496 | pctx.PrefixedExistentPathsForSourcesVariable("CommonGlobalIncludes", "-I", commonGlobalIncludes) |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 497 | |
Cole Faust | 8982b1c | 2024-04-08 16:54:45 -0700 | [diff] [blame] | 498 | pctx.StaticVariable("CLANG_DEFAULT_VERSION", ClangDefaultVersion) |
| 499 | pctx.StaticVariable("CLANG_DEFAULT_SHORT_VERSION", ClangDefaultShortVersion) |
Sam Delmerico | 3eda019 | 2023-04-14 18:07:25 +0000 | [diff] [blame] | 500 | |
Dan Willemsen | 9fe1410 | 2021-07-13 21:52:04 -0700 | [diff] [blame] | 501 | pctx.StaticVariableWithEnvOverride("ClangBase", "LLVM_PREBUILTS_BASE", ClangDefaultBase) |
Sam Delmerico | 3eda019 | 2023-04-14 18:07:25 +0000 | [diff] [blame] | 502 | pctx.StaticVariableWithEnvOverride("ClangVersion", "LLVM_PREBUILTS_VERSION", ClangDefaultVersion) |
Colin Cross | b98c8b0 | 2016-07-29 13:44:28 -0700 | [diff] [blame] | 503 | pctx.StaticVariable("ClangPath", "${ClangBase}/${HostPrebuiltTag}/${ClangVersion}") |
| 504 | pctx.StaticVariable("ClangBin", "${ClangPath}/bin") |
| 505 | |
Sam Delmerico | 3eda019 | 2023-04-14 18:07:25 +0000 | [diff] [blame] | 506 | pctx.StaticVariableWithEnvOverride("ClangShortVersion", "LLVM_RELEASE_VERSION", ClangDefaultShortVersion) |
Yi Kong | bd18881 | 2023-02-08 19:51:59 +0900 | [diff] [blame] | 507 | pctx.StaticVariable("ClangAsanLibDir", "${ClangBase}/linux-x86/${ClangVersion}/lib/clang/${ClangShortVersion}/lib/linux") |
Alistair Strachan | 777475c | 2016-08-26 12:55:49 -0700 | [diff] [blame] | 508 | |
Cole Faust | 8982b1c | 2024-04-08 16:54:45 -0700 | [diff] [blame] | 509 | pctx.StaticVariable("WarningAllowedProjects", strings.Join(WarningAllowedProjects, " ")) |
Sam Delmerico | 036afab | 2023-05-04 16:43:36 -0400 | [diff] [blame] | 510 | |
Jayant Chowdhary | e622d20 | 2017-02-01 19:19:52 -0800 | [diff] [blame] | 511 | // These are tied to the version of LLVM directly in external/llvm, so they might trail the host prebuilts |
| 512 | // being used for the rest of the build process. |
| 513 | pctx.SourcePathVariable("RSClangBase", "prebuilts/clang/host") |
| 514 | pctx.SourcePathVariable("RSClangVersion", "clang-3289846") |
| 515 | pctx.SourcePathVariable("RSReleaseVersion", "3.8") |
| 516 | pctx.StaticVariable("RSLLVMPrebuiltsPath", "${RSClangBase}/${HostPrebuiltTag}/${RSClangVersion}/bin") |
| 517 | pctx.StaticVariable("RSIncludePath", "${RSLLVMPrebuiltsPath}/../lib64/clang/${RSReleaseVersion}/include") |
| 518 | |
Alix | e266787 | 2023-04-24 14:57:32 +0000 | [diff] [blame] | 519 | rsGlobalIncludes := []string{ |
| 520 | "external/clang/lib/Headers", |
| 521 | "frameworks/rs/script_api/include", |
| 522 | } |
| 523 | pctx.PrefixedExistentPathsForSourcesVariable("RsGlobalIncludes", "-I", rsGlobalIncludes) |
Colin Cross | 2a252be | 2017-05-01 17:37:24 -0700 | [diff] [blame] | 524 | |
Dan Willemsen | 54daaf0 | 2018-03-12 13:24:09 -0700 | [diff] [blame] | 525 | pctx.VariableFunc("CcWrapper", func(ctx android.PackageVarContext) string { |
| 526 | if override := ctx.Config().Getenv("CC_WRAPPER"); override != "" { |
| 527 | return override + " " |
Alistair Strachan | 777475c | 2016-08-26 12:55:49 -0700 | [diff] [blame] | 528 | } |
Dan Willemsen | 54daaf0 | 2018-03-12 13:24:09 -0700 | [diff] [blame] | 529 | return "" |
Alistair Strachan | 777475c | 2016-08-26 12:55:49 -0700 | [diff] [blame] | 530 | }) |
Ramy Medhat | 9a90fe5 | 2020-04-13 13:21:23 -0400 | [diff] [blame] | 531 | |
Colin Cross | 77cdcfd | 2021-03-12 11:28:25 -0800 | [diff] [blame] | 532 | pctx.StaticVariableWithEnvOverride("RECXXPool", "RBE_CXX_POOL", remoteexec.DefaultPool) |
| 533 | pctx.StaticVariableWithEnvOverride("RECXXLinksPool", "RBE_CXX_LINKS_POOL", remoteexec.DefaultPool) |
| 534 | pctx.StaticVariableWithEnvOverride("REClangTidyPool", "RBE_CLANG_TIDY_POOL", remoteexec.DefaultPool) |
| 535 | pctx.StaticVariableWithEnvOverride("RECXXLinksExecStrategy", "RBE_CXX_LINKS_EXEC_STRATEGY", remoteexec.LocalExecStrategy) |
| 536 | pctx.StaticVariableWithEnvOverride("REClangTidyExecStrategy", "RBE_CLANG_TIDY_EXEC_STRATEGY", remoteexec.LocalExecStrategy) |
| 537 | pctx.StaticVariableWithEnvOverride("REAbiDumperExecStrategy", "RBE_ABI_DUMPER_EXEC_STRATEGY", remoteexec.LocalExecStrategy) |
| 538 | pctx.StaticVariableWithEnvOverride("REAbiLinkerExecStrategy", "RBE_ABI_LINKER_EXEC_STRATEGY", remoteexec.LocalExecStrategy) |
Colin Cross | 4d9c2d1 | 2016-07-29 12:48:20 -0700 | [diff] [blame] | 539 | } |
| 540 | |
Cole Faust | 8982b1c | 2024-04-08 16:54:45 -0700 | [diff] [blame] | 541 | var HostPrebuiltTag = pctx.VariableConfigMethod("HostPrebuiltTag", android.Config.PrebuiltOS) |
Dan Willemsen | 9fe1410 | 2021-07-13 21:52:04 -0700 | [diff] [blame] | 542 | |
| 543 | func ClangPath(ctx android.PathContext, file string) android.SourcePath { |
| 544 | type clangToolKey string |
| 545 | |
| 546 | key := android.NewCustomOnceKey(clangToolKey(file)) |
| 547 | |
| 548 | return ctx.Config().OnceSourcePath(key, func() android.SourcePath { |
| 549 | return clangPath(ctx).Join(ctx, file) |
| 550 | }) |
| 551 | } |
| 552 | |
| 553 | var clangPathKey = android.NewOnceKey("clangPath") |
| 554 | |
| 555 | func clangPath(ctx android.PathContext) android.SourcePath { |
| 556 | return ctx.Config().OnceSourcePath(clangPathKey, func() android.SourcePath { |
| 557 | clangBase := ClangDefaultBase |
| 558 | if override := ctx.Config().Getenv("LLVM_PREBUILTS_BASE"); override != "" { |
| 559 | clangBase = override |
| 560 | } |
| 561 | clangVersion := ClangDefaultVersion |
| 562 | if override := ctx.Config().Getenv("LLVM_PREBUILTS_VERSION"); override != "" { |
| 563 | clangVersion = override |
| 564 | } |
| 565 | return android.PathForSource(ctx, clangBase, ctx.Config().PrebuiltOS(), clangVersion) |
| 566 | }) |
| 567 | } |