| bootstrap_go_package { |
| name: "soong-art", |
| pkgPath: "android/soong/art", |
| deps: [ |
| "blueprint", |
| "blueprint-pathtools", |
| "blueprint-proptools", |
| "soong", |
| "soong-android", |
| "soong-apex", |
| "soong-cc", |
| ], |
| srcs: [ |
| "art.go", |
| "codegen.go", |
| "makevars.go", |
| ], |
| pluginFor: ["soong_build"], |
| } |
| |
| art_clang_tidy_errors = [ |
| "android-cloexec-dup", |
| "android-cloexec-open", |
| "bugprone-argument-comment", |
| "bugprone-lambda-function-name", |
| "bugprone-unused-raii", // Protect scoped things like MutexLock. |
| "bugprone-unused-return-value", |
| "bugprone-virtual-near-miss", |
| "modernize-use-bool-literals", |
| "modernize-use-nullptr", |
| "modernize-use-using", |
| "performance-faster-string-find", |
| "performance-for-range-copy", |
| "performance-implicit-conversion-in-loop", |
| "performance-noexcept-move-constructor", |
| "performance-unnecessary-copy-initialization", |
| "performance-unnecessary-value-param", |
| "misc-unused-using-decls", |
| ] |
| |
| art_clang_tidy_disabled = [ |
| "-google-default-arguments", |
| // We have local stores that are only used for debug checks. |
| "-clang-analyzer-deadcode.DeadStores", |
| // We are OK with some static globals and that they can, in theory, throw. |
| "-cert-err58-cpp", |
| // We have lots of C-style variadic functions, and are OK with them. JNI ensures |
| // that working around this warning would be extra-painful. |
| "-cert-dcl50-cpp", |
| // "Modernization" we don't agree with. |
| "-modernize-use-auto", |
| "-modernize-return-braced-init-list", |
| "-modernize-use-default-member-init", |
| "-modernize-pass-by-value", |
| ] |
| |
| art_global_defaults { |
| // Additional flags are computed by art.go |
| |
| name: "art_defaults", |
| |
| // This is the default visibility for the //art package, but we repeat it |
| // here so that it gets merged with other visibility rules in modules |
| // extending these defaults. |
| visibility: ["//art:__subpackages__"], |
| |
| cflags: [ |
| // Base set of cflags used by all things ART. |
| "-fno-rtti", |
| "-ggdb3", |
| "-Wall", |
| "-Werror", |
| "-Wextra", |
| "-Wstrict-aliasing", |
| "-fstrict-aliasing", |
| "-Wunreachable-code", |
| "-Wredundant-decls", |
| "-Wshadow", |
| "-Wunused", |
| "-fvisibility=protected", |
| |
| // Warn about thread safety violations with clang. |
| "-Wthread-safety", |
| // TODO(b/144045034): turn on -Wthread-safety-negative |
| //"-Wthread-safety-negative", |
| |
| // Warn if switch fallthroughs aren't annotated. |
| "-Wimplicit-fallthrough", |
| |
| // Enable float equality warnings. |
| "-Wfloat-equal", |
| |
| // Enable warning of converting ints to void*. |
| "-Wint-to-void-pointer-cast", |
| |
| // Enable warning of wrong unused annotations. |
| "-Wused-but-marked-unused", |
| |
| // Enable warning for deprecated language features. |
| "-Wdeprecated", |
| |
| // Enable warning for unreachable break & return. |
| "-Wunreachable-code-break", |
| "-Wunreachable-code-return", |
| |
| // Disable warning for use of offsetof on non-standard layout type. |
| // We use it to implement OFFSETOF_MEMBER - see macros.h. |
| "-Wno-invalid-offsetof", |
| |
| // Enable inconsistent-missing-override warning. This warning is disabled by default in |
| // Android. |
| "-Winconsistent-missing-override", |
| |
| // Enable thread annotations for std::mutex, etc. |
| "-D_LIBCPP_ENABLE_THREAD_SAFETY_ANNOTATIONS", |
| ], |
| |
| arch: { |
| x86: { |
| avx2: { |
| cflags: [ |
| "-mavx2", |
| "-mfma", |
| ], |
| }, |
| }, |
| x86_64: { |
| avx2: { |
| cflags: [ |
| "-mavx2", |
| "-mfma", |
| ], |
| }, |
| }, |
| }, |
| |
| target: { |
| android: { |
| cflags: [ |
| "-DART_TARGET", |
| |
| // To use oprofile_android --callgraph, uncomment this and recompile with |
| // mmma -j art |
| // "-fno-omit-frame-pointer", |
| // "-marm", |
| // "-mapcs", |
| ], |
| header_libs: [ |
| // We optimize Thread::Current() with a direct TLS access. This requires access to a |
| // platform specific Bionic header. |
| "bionic_libc_platform_headers", |
| ], |
| }, |
| linux: { |
| cflags: [ |
| // Enable missing-noreturn only on non-Mac. As lots of things are not implemented for |
| // Apple, it's a pain. |
| "-Wmissing-noreturn", |
| ], |
| }, |
| linux_bionic: { |
| header_libs: [ |
| // We optimize Thread::Current() with a direct TLS access. This requires access to a |
| // platform specific Bionic header. |
| "bionic_libc_platform_headers", |
| ], |
| strip: { |
| // Do not strip art libs when building for linux-bionic. |
| // Otherwise we can't get any symbols out of crashes. |
| none: true, |
| }, |
| }, |
| darwin: { |
| enabled: false, |
| }, |
| host: { |
| cflags: [ |
| // Bug: 15446488. We don't omit the frame pointer to work around |
| // clang/libunwind bugs that cause SEGVs in run-test-004-ThreadStress. |
| "-fno-omit-frame-pointer", |
| // The build assumes that all our x86/x86_64 hosts (such as buildbots and developer |
| // desktops) support at least sse4.2/popcount. This firstly implies that the ART |
| // runtime binary itself may exploit these features. Secondly, this implies that |
| // the ART runtime passes these feature flags to dex2oat and JIT by calling the |
| // method InstructionSetFeatures::FromCppDefines(). Since invoking dex2oat directly |
| // does not pick up these flags, cross-compiling from a x86/x86_64 host to a |
| // x86/x86_64 target should not be affected. |
| "-msse4.2", |
| "-mpopcnt", |
| ], |
| }, |
| }, |
| |
| codegen: { |
| arm: { |
| cflags: ["-DART_ENABLE_CODEGEN_arm"], |
| }, |
| arm64: { |
| cflags: ["-DART_ENABLE_CODEGEN_arm64"], |
| }, |
| x86: { |
| cflags: ["-DART_ENABLE_CODEGEN_x86"], |
| }, |
| x86_64: { |
| cflags: ["-DART_ENABLE_CODEGEN_x86_64"], |
| }, |
| }, |
| |
| tidy_checks: art_clang_tidy_errors + art_clang_tidy_disabled, |
| tidy_checks_as_errors: art_clang_tidy_errors, |
| |
| tidy_flags: [ |
| // The static analyzer treats DCHECK as always enabled; we sometimes get |
| // false positives when we use DCHECKs with code that relies on NDEBUG. |
| "-extra-arg=-UNDEBUG", |
| // clang-tidy complains about functions like: |
| // void foo() { CHECK(kIsFooEnabled); /* do foo... */ } |
| // not being marked noreturn if kIsFooEnabled is false. |
| "-extra-arg=-Wno-missing-noreturn", |
| // Because tidy doesn't like our flow checks for compile-time configuration and thinks that |
| // the following code is dead (it is, but not for all configurations), disable unreachable |
| // code detection in Clang for tidy builds. It is still on for regular build steps, so we |
| // will still get the "real" errors. |
| "-extra-arg=-Wno-unreachable-code", |
| ], |
| } |
| |
| art_debug_defaults { |
| name: "art_debug_defaults", |
| visibility: ["//art:__subpackages__"], |
| cflags: [ |
| "-DDYNAMIC_ANNOTATIONS_ENABLED=1", |
| "-DVIXL_DEBUG", |
| "-UNDEBUG", |
| ], |
| asflags: [ |
| "-UNDEBUG", |
| ], |
| target: { |
| // This has to be duplicated for android and host to make sure it |
| // comes after the -Wframe-larger-than warnings inserted by art.go |
| // target-specific properties |
| android: { |
| cflags: ["-Wno-frame-larger-than="], |
| }, |
| host: { |
| cflags: ["-Wno-frame-larger-than="], |
| }, |
| }, |
| } |