0c66bc615b
Replace the android.BuildOs constant with Config.BuildOS so that it can vary based on the product config. Bug: 190084016 Test: all Soong tests Change-Id: Ia67f872d8b2ab788747a22e3a9659dc21c9775cd
411 lines
14 KiB
Go
411 lines
14 KiB
Go
// Copyright 2016 Google Inc. All rights reserved.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package config
|
|
|
|
import (
|
|
"runtime"
|
|
"strings"
|
|
|
|
"android/soong/android"
|
|
"android/soong/remoteexec"
|
|
)
|
|
|
|
var (
|
|
// Flags used by lots of devices. Putting them in package static variables
|
|
// will save bytes in build.ninja so they aren't repeated for every file
|
|
commonGlobalCflags = []string{
|
|
"-DANDROID",
|
|
"-fmessage-length=0",
|
|
"-W",
|
|
"-Wall",
|
|
"-Wno-unused",
|
|
"-Winit-self",
|
|
"-Wpointer-arith",
|
|
"-Wunreachable-code-loop-increment",
|
|
|
|
// Make paths in deps files relative
|
|
"-no-canonical-prefixes",
|
|
|
|
"-DNDEBUG",
|
|
"-UDEBUG",
|
|
|
|
"-fno-exceptions",
|
|
"-Wno-multichar",
|
|
|
|
"-O2",
|
|
"-g",
|
|
"-fdebug-default-version=5",
|
|
"-fdebug-info-for-profiling",
|
|
|
|
"-fno-strict-aliasing",
|
|
|
|
"-Werror=date-time",
|
|
"-Werror=pragma-pack",
|
|
"-Werror=pragma-pack-suspicious-include",
|
|
"-Werror=string-plus-int",
|
|
"-Werror=unreachable-code-loop-increment",
|
|
|
|
"-D__compiler_offsetof=__builtin_offsetof",
|
|
|
|
// Emit address-significance table which allows linker to perform safe ICF. Clang does
|
|
// not emit the table by default on Android since NDK still uses GNU binutils.
|
|
"-faddrsig",
|
|
|
|
// Turn on -fcommon explicitly, since Clang now defaults to -fno-common. The cleanup bug
|
|
// tracking this is http://b/151457797.
|
|
"-fcommon",
|
|
|
|
// Help catch common 32/64-bit errors.
|
|
"-Werror=int-conversion",
|
|
|
|
// Enable the new pass manager.
|
|
"-fexperimental-new-pass-manager",
|
|
|
|
// Disable overly aggressive warning for macros defined with a leading underscore
|
|
// This happens in AndroidConfig.h, which is included nearly everywhere.
|
|
// TODO: can we remove this now?
|
|
"-Wno-reserved-id-macro",
|
|
|
|
// Workaround for ccache with clang.
|
|
// See http://petereisentraut.blogspot.com/2011/05/ccache-and-clang.html.
|
|
"-Wno-unused-command-line-argument",
|
|
|
|
// Force clang to always output color diagnostics. Ninja will strip the ANSI
|
|
// color codes if it is not running in a terminal.
|
|
"-fcolor-diagnostics",
|
|
|
|
// Warnings from clang-7.0
|
|
"-Wno-sign-compare",
|
|
|
|
// Warnings from clang-8.0
|
|
"-Wno-defaulted-function-deleted",
|
|
|
|
// Disable -Winconsistent-missing-override until we can clean up the existing
|
|
// codebase for it.
|
|
"-Wno-inconsistent-missing-override",
|
|
|
|
// Warnings from clang-10
|
|
// Nested and array designated initialization is nice to have.
|
|
"-Wno-c99-designator",
|
|
|
|
// Warnings from clang-12
|
|
"-Wno-gnu-folding-constant",
|
|
|
|
// Calls to the APIs that are newer than the min sdk version of the caller should be
|
|
// guarded with __builtin_available.
|
|
"-Wunguarded-availability",
|
|
// This macro allows the bionic versioning.h to indirectly determine whether the
|
|
// option -Wunguarded-availability is on or not.
|
|
"-D__ANDROID_UNAVAILABLE_SYMBOLS_ARE_WEAK__",
|
|
}
|
|
|
|
commonGlobalConlyflags = []string{}
|
|
|
|
deviceGlobalCflags = []string{
|
|
"-ffunction-sections",
|
|
"-fdata-sections",
|
|
"-fno-short-enums",
|
|
"-funwind-tables",
|
|
"-fstack-protector-strong",
|
|
"-Wa,--noexecstack",
|
|
"-D_FORTIFY_SOURCE=2",
|
|
|
|
"-Wstrict-aliasing=2",
|
|
|
|
"-Werror=return-type",
|
|
"-Werror=non-virtual-dtor",
|
|
"-Werror=address",
|
|
"-Werror=sequence-point",
|
|
"-Werror=format-security",
|
|
"-nostdlibinc",
|
|
}
|
|
|
|
deviceGlobalCppflags = []string{
|
|
"-fvisibility-inlines-hidden",
|
|
}
|
|
|
|
deviceGlobalLdflags = []string{
|
|
"-Wl,-z,noexecstack",
|
|
"-Wl,-z,relro",
|
|
"-Wl,-z,now",
|
|
"-Wl,--build-id=md5",
|
|
"-Wl,--warn-shared-textrel",
|
|
"-Wl,--fatal-warnings",
|
|
"-Wl,--no-undefined-version",
|
|
// TODO: Eventually we should link against a libunwind.a with hidden symbols, and then these
|
|
// --exclude-libs arguments can be removed.
|
|
"-Wl,--exclude-libs,libgcc.a",
|
|
"-Wl,--exclude-libs,libgcc_stripped.a",
|
|
"-Wl,--exclude-libs,libunwind_llvm.a",
|
|
"-Wl,--exclude-libs,libunwind.a",
|
|
"-Wl,--icf=safe",
|
|
}
|
|
|
|
deviceGlobalLldflags = append(deviceGlobalLdflags,
|
|
[]string{
|
|
"-fuse-ld=lld",
|
|
}...)
|
|
|
|
hostGlobalCflags = []string{}
|
|
|
|
hostGlobalCppflags = []string{}
|
|
|
|
hostGlobalLdflags = []string{}
|
|
|
|
hostGlobalLldflags = []string{"-fuse-ld=lld"}
|
|
|
|
commonGlobalCppflags = []string{
|
|
"-Wsign-promo",
|
|
|
|
// -Wimplicit-fallthrough is not enabled by -Wall.
|
|
"-Wimplicit-fallthrough",
|
|
|
|
// Enable clang's thread-safety annotations in libcxx.
|
|
"-D_LIBCPP_ENABLE_THREAD_SAFETY_ANNOTATIONS",
|
|
|
|
// libc++'s math.h has an #include_next outside of system_headers.
|
|
"-Wno-gnu-include-next",
|
|
}
|
|
|
|
noOverrideGlobalCflags = []string{
|
|
"-Werror=bool-operation",
|
|
"-Werror=implicit-int-float-conversion",
|
|
"-Werror=int-in-bool-context",
|
|
"-Werror=int-to-pointer-cast",
|
|
"-Werror=pointer-to-int-cast",
|
|
"-Werror=string-compare",
|
|
"-Werror=xor-used-as-pow",
|
|
// http://b/161386391 for -Wno-void-pointer-to-enum-cast
|
|
"-Wno-void-pointer-to-enum-cast",
|
|
// http://b/161386391 for -Wno-void-pointer-to-int-cast
|
|
"-Wno-void-pointer-to-int-cast",
|
|
// http://b/161386391 for -Wno-pointer-to-int-cast
|
|
"-Wno-pointer-to-int-cast",
|
|
"-Werror=fortify-source",
|
|
|
|
"-Werror=address-of-temporary",
|
|
// Bug: http://b/29823425 Disable -Wnull-dereference until the
|
|
// new cases detected by this warning in Clang r271374 are
|
|
// fixed.
|
|
//"-Werror=null-dereference",
|
|
"-Werror=return-type",
|
|
|
|
// http://b/72331526 Disable -Wtautological-* until the instances detected by these
|
|
// new warnings are fixed.
|
|
"-Wno-tautological-constant-compare",
|
|
"-Wno-tautological-type-limit-compare",
|
|
// http://b/145210666
|
|
"-Wno-reorder-init-list",
|
|
// http://b/145211066
|
|
"-Wno-implicit-int-float-conversion",
|
|
// New warnings to be fixed after clang-r377782.
|
|
"-Wno-int-in-bool-context", // http://b/148287349
|
|
"-Wno-sizeof-array-div", // http://b/148815709
|
|
"-Wno-tautological-overlap-compare", // http://b/148815696
|
|
// New warnings to be fixed after clang-r383902.
|
|
"-Wno-deprecated-copy", // http://b/153746672
|
|
"-Wno-range-loop-construct", // http://b/153747076
|
|
"-Wno-misleading-indentation", // http://b/153746954
|
|
"-Wno-zero-as-null-pointer-constant", // http://b/68236239
|
|
"-Wno-deprecated-anon-enum-enum-conversion", // http://b/153746485
|
|
"-Wno-deprecated-enum-enum-conversion", // http://b/153746563
|
|
"-Wno-string-compare", // http://b/153764102
|
|
"-Wno-enum-enum-conversion", // http://b/154138986
|
|
"-Wno-enum-float-conversion", // http://b/154255917
|
|
"-Wno-pessimizing-move", // http://b/154270751
|
|
// New warnings to be fixed after clang-r399163
|
|
"-Wno-non-c-typedef-for-linkage", // http://b/161304145
|
|
// New warnings to be fixed after clang-r407598
|
|
"-Wno-string-concatenation", // http://b/175068488
|
|
}
|
|
|
|
// Extra cflags for external third-party projects to disable warnings that
|
|
// are infeasible to fix in all the external projects and their upstream repos.
|
|
extraExternalCflags = []string{
|
|
"-Wno-enum-compare",
|
|
"-Wno-enum-compare-switch",
|
|
|
|
// http://b/72331524 Allow null pointer arithmetic until the instances detected by
|
|
// this new warning are fixed.
|
|
"-Wno-null-pointer-arithmetic",
|
|
|
|
// Bug: http://b/29823425 Disable -Wnull-dereference until the
|
|
// new instances detected by this warning are fixed.
|
|
"-Wno-null-dereference",
|
|
|
|
// http://b/145211477
|
|
"-Wno-pointer-compare",
|
|
// http://b/145211022
|
|
"-Wno-xor-used-as-pow",
|
|
// http://b/145211022
|
|
"-Wno-final-dtor-non-final-class",
|
|
|
|
// http://b/165945989
|
|
"-Wno-psabi",
|
|
}
|
|
|
|
IllegalFlags = []string{
|
|
"-w",
|
|
}
|
|
|
|
CStdVersion = "gnu99"
|
|
CppStdVersion = "gnu++17"
|
|
ExperimentalCStdVersion = "gnu11"
|
|
ExperimentalCppStdVersion = "gnu++2a"
|
|
|
|
// prebuilts/clang default settings.
|
|
ClangDefaultBase = "prebuilts/clang/host"
|
|
ClangDefaultVersion = "clang-r416183b1"
|
|
ClangDefaultShortVersion = "12.0.7"
|
|
|
|
// Directories with warnings from Android.bp files.
|
|
WarningAllowedProjects = []string{
|
|
"device/",
|
|
"vendor/",
|
|
}
|
|
|
|
// Directories with warnings from Android.mk files.
|
|
WarningAllowedOldProjects = []string{}
|
|
)
|
|
|
|
var pctx = android.NewPackageContext("android/soong/cc/config")
|
|
|
|
func init() {
|
|
if runtime.GOOS == "linux" {
|
|
commonGlobalCflags = append(commonGlobalCflags, "-fdebug-prefix-map=/proc/self/cwd=")
|
|
}
|
|
|
|
exportStringListStaticVariable("CommonGlobalConlyflags", commonGlobalConlyflags)
|
|
exportStringListStaticVariable("DeviceGlobalCppflags", deviceGlobalCppflags)
|
|
exportStringListStaticVariable("DeviceGlobalLdflags", deviceGlobalLdflags)
|
|
exportStringListStaticVariable("DeviceGlobalLldflags", deviceGlobalLldflags)
|
|
exportStringListStaticVariable("HostGlobalCppflags", hostGlobalCppflags)
|
|
exportStringListStaticVariable("HostGlobalLdflags", hostGlobalLdflags)
|
|
exportStringListStaticVariable("HostGlobalLldflags", hostGlobalLldflags)
|
|
|
|
// Export the static default CommonGlobalCflags to Bazel.
|
|
// TODO(187086342): handle cflags that are set in VariableFuncs.
|
|
bazelCommonGlobalCflags := append(
|
|
commonGlobalCflags,
|
|
[]string{
|
|
// Default to zero initialization.
|
|
"-ftrivial-auto-var-init=zero",
|
|
"-enable-trivial-auto-var-init-zero-knowing-it-will-be-removed-from-clang",
|
|
}...)
|
|
exportedStringListVars.Set("CommonGlobalCflags", bazelCommonGlobalCflags)
|
|
|
|
pctx.VariableFunc("CommonGlobalCflags", func(ctx android.PackageVarContext) string {
|
|
flags := commonGlobalCflags
|
|
|
|
// http://b/131390872
|
|
// Automatically initialize any uninitialized stack variables.
|
|
// Prefer zero-init if multiple options are set.
|
|
if ctx.Config().IsEnvTrue("AUTO_ZERO_INITIALIZE") {
|
|
flags = append(flags, "-ftrivial-auto-var-init=zero -enable-trivial-auto-var-init-zero-knowing-it-will-be-removed-from-clang")
|
|
} else if ctx.Config().IsEnvTrue("AUTO_PATTERN_INITIALIZE") {
|
|
flags = append(flags, "-ftrivial-auto-var-init=pattern")
|
|
} else if ctx.Config().IsEnvTrue("AUTO_UNINITIALIZE") {
|
|
flags = append(flags, "-ftrivial-auto-var-init=uninitialized")
|
|
} else {
|
|
// Default to zero initialization.
|
|
flags = append(flags, "-ftrivial-auto-var-init=zero -enable-trivial-auto-var-init-zero-knowing-it-will-be-removed-from-clang")
|
|
}
|
|
return strings.Join(flags, " ")
|
|
})
|
|
|
|
// Export the static default DeviceGlobalCflags to Bazel.
|
|
// TODO(187086342): handle cflags that are set in VariableFuncs.
|
|
exportedStringListVars.Set("DeviceGlobalCflags", deviceGlobalCflags)
|
|
|
|
pctx.VariableFunc("DeviceGlobalCflags", func(ctx android.PackageVarContext) string {
|
|
return strings.Join(deviceGlobalCflags, " ")
|
|
})
|
|
|
|
exportStringListStaticVariable("HostGlobalCflags", hostGlobalCflags)
|
|
exportStringListStaticVariable("NoOverrideGlobalCflags", noOverrideGlobalCflags)
|
|
exportStringListStaticVariable("CommonGlobalCppflags", commonGlobalCppflags)
|
|
exportStringListStaticVariable("ExternalCflags", extraExternalCflags)
|
|
|
|
// Everything in these lists is a crime against abstraction and dependency tracking.
|
|
// Do not add anything to this list.
|
|
commonGlobalIncludes := []string{
|
|
"system/core/include",
|
|
"system/logging/liblog/include",
|
|
"system/media/audio/include",
|
|
"hardware/libhardware/include",
|
|
"hardware/libhardware_legacy/include",
|
|
"hardware/ril/include",
|
|
"frameworks/native/include",
|
|
"frameworks/native/opengl/include",
|
|
"frameworks/av/include",
|
|
}
|
|
exportedStringListVars.Set("CommonGlobalIncludes", commonGlobalIncludes)
|
|
pctx.PrefixedExistentPathsForSourcesVariable("CommonGlobalIncludes", "-I", commonGlobalIncludes)
|
|
|
|
pctx.SourcePathVariable("ClangDefaultBase", ClangDefaultBase)
|
|
pctx.VariableFunc("ClangBase", func(ctx android.PackageVarContext) string {
|
|
if override := ctx.Config().Getenv("LLVM_PREBUILTS_BASE"); override != "" {
|
|
return override
|
|
}
|
|
return "${ClangDefaultBase}"
|
|
})
|
|
pctx.VariableFunc("ClangVersion", func(ctx android.PackageVarContext) string {
|
|
if override := ctx.Config().Getenv("LLVM_PREBUILTS_VERSION"); override != "" {
|
|
return override
|
|
}
|
|
return ClangDefaultVersion
|
|
})
|
|
pctx.StaticVariable("ClangPath", "${ClangBase}/${HostPrebuiltTag}/${ClangVersion}")
|
|
pctx.StaticVariable("ClangBin", "${ClangPath}/bin")
|
|
|
|
pctx.VariableFunc("ClangShortVersion", func(ctx android.PackageVarContext) string {
|
|
if override := ctx.Config().Getenv("LLVM_RELEASE_VERSION"); override != "" {
|
|
return override
|
|
}
|
|
return ClangDefaultShortVersion
|
|
})
|
|
pctx.StaticVariable("ClangAsanLibDir", "${ClangBase}/linux-x86/${ClangVersion}/lib64/clang/${ClangShortVersion}/lib/linux")
|
|
|
|
// These are tied to the version of LLVM directly in external/llvm, so they might trail the host prebuilts
|
|
// being used for the rest of the build process.
|
|
pctx.SourcePathVariable("RSClangBase", "prebuilts/clang/host")
|
|
pctx.SourcePathVariable("RSClangVersion", "clang-3289846")
|
|
pctx.SourcePathVariable("RSReleaseVersion", "3.8")
|
|
pctx.StaticVariable("RSLLVMPrebuiltsPath", "${RSClangBase}/${HostPrebuiltTag}/${RSClangVersion}/bin")
|
|
pctx.StaticVariable("RSIncludePath", "${RSLLVMPrebuiltsPath}/../lib64/clang/${RSReleaseVersion}/include")
|
|
|
|
pctx.PrefixedExistentPathsForSourcesVariable("RsGlobalIncludes", "-I",
|
|
[]string{
|
|
"external/clang/lib/Headers",
|
|
"frameworks/rs/script_api/include",
|
|
})
|
|
|
|
pctx.VariableFunc("CcWrapper", func(ctx android.PackageVarContext) string {
|
|
if override := ctx.Config().Getenv("CC_WRAPPER"); override != "" {
|
|
return override + " "
|
|
}
|
|
return ""
|
|
})
|
|
|
|
pctx.StaticVariableWithEnvOverride("RECXXPool", "RBE_CXX_POOL", remoteexec.DefaultPool)
|
|
pctx.StaticVariableWithEnvOverride("RECXXLinksPool", "RBE_CXX_LINKS_POOL", remoteexec.DefaultPool)
|
|
pctx.StaticVariableWithEnvOverride("REClangTidyPool", "RBE_CLANG_TIDY_POOL", remoteexec.DefaultPool)
|
|
pctx.StaticVariableWithEnvOverride("RECXXLinksExecStrategy", "RBE_CXX_LINKS_EXEC_STRATEGY", remoteexec.LocalExecStrategy)
|
|
pctx.StaticVariableWithEnvOverride("REClangTidyExecStrategy", "RBE_CLANG_TIDY_EXEC_STRATEGY", remoteexec.LocalExecStrategy)
|
|
pctx.StaticVariableWithEnvOverride("REAbiDumperExecStrategy", "RBE_ABI_DUMPER_EXEC_STRATEGY", remoteexec.LocalExecStrategy)
|
|
pctx.StaticVariableWithEnvOverride("REAbiLinkerExecStrategy", "RBE_ABI_LINKER_EXEC_STRATEGY", remoteexec.LocalExecStrategy)
|
|
}
|
|
|
|
var HostPrebuiltTag = pctx.VariableConfigMethod("HostPrebuiltTag", android.Config.PrebuiltOS)
|